Skip to content

Add R6 to Suggests - #81

Open
MichaelChirico wants to merge 1 commit into
dselivanov:masterfrom
MichaelChirico:patch-1
Open

Add R6 to Suggests#81
MichaelChirico wants to merge 1 commit into
dselivanov:masterfrom
MichaelChirico:patch-1

Conversation

@MichaelChirico

Copy link
Copy Markdown
Collaborator

Closes #80

R doesn't have a good system for requiring such dependencies, I think Suggests is the best compromise within the current system.

@dselivanov

Copy link
Copy Markdown
Owner

@MichaelChirico apologise, forgot about this PR. I;ve sent collaborator invites to you and @david-cortes to make things more fluent here as I'm lacing time.
There is R-cmd-check error. Is this because env is too old? I'm happy to merge once resolved. Apologise for delay once again

@MichaelChirico

Copy link
Copy Markdown
Collaborator Author

You have:

Depends: R (>= 3.6.0), methods, Matrix (>= 1.3)

But Matrix itself requires R 4.4:

https://github.com/cran/Matrix/blob/96f735418be2cd873ebdb2d57935d5ad76cdc0b7/DESCRIPTION#L40

So testing on 3.6.0 won't work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build-time dependency on R6 is not declared

2 participants