Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Note about missingArg not being found in baseR but rather 'methods' #107

Closed
selkamand opened this issue Feb 8, 2024 · 0 comments · Fixed by #108
Closed

Note about missingArg not being found in baseR but rather 'methods' #107

selkamand opened this issue Feb 8, 2024 · 0 comments · Fixed by #108

Comments

@selkamand
Copy link
Owner

 required_args_are_missing : <anonymous>: no visible global function
    definition for ‘missingArg’
  Undefined global functions or variables:
    missingArg
  Consider adding
    importFrom("methods", "missingArg")
  to your NAMESPACE file (and ensure that your DESCRIPTION Imports field
  contains 'methods').

Easy fix is just to add explicit namespace methods::missingArg and import the methods package as a 'dependency'

Its not truly a dependency since methods is one of the base R packages (distributed with base R) so I don't mind this at all.

Otherwise we could just switch to an Rlang method for accomplishing the same thing. But if i had to bet - i suspect methods::missingArg is more likely to be stable over time than an equivalent Rlang solution so will go with this

@selkamand selkamand linked a pull request Feb 8, 2024 that will close this issue
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 a pull request may close this issue.

1 participant