We only need to suggest methods, not import it

[hadley]

No description provided.

[cderv]

We call methods::show in here

evaluate/R/output.R

Line 48 in c750df4

render <- function(x) if (isS4(x)) methods::show(x) else print(x)

is this possible that the methods package is not available ?

Or we should check also for it being installed if we want to keep Suggest

- render <- function(x) if (isS4(x)) methods::show(x) else print(x)
+ render <- function(x) if (isS4(x) && requireNamespace("methods", quietly = TRUE)) methods::show(x) else print(x)

This is the practice I follow with Suggest

[hadley]

I don't think it's possible for methods to not be available and to have an S4 object. (i.e. if isS4(x) is TRUE, it implies that methods must be available).

[cderv]

(i.e. if isS4(x) is TRUE, it implies that methods must be available).

Then it is ok I guess. 👍