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

CRAN issue, ... in @param not in function #58

Closed
khufkens opened this issue Feb 3, 2020 · 4 comments
Closed

CRAN issue, ... in @param not in function #58

khufkens opened this issue Feb 3, 2020 · 4 comments

Comments

@khufkens
Copy link
Member

khufkens commented Feb 3, 2020

Not sure why this is detected right now and not like x months ago as nothing changed.

Anyway, @param here should be removed.

https://github.com/khufkens/ecmwfr/blob/60d2b35fc1fd1eaf92505dea5e7b9db36ae79ed3/R/wf_transfer.R#L13

Dear maintainer,

Please see the problems shown on
<https://cran.r-project.org/web/checks/check_results_ecmwfr.html>.

Specifically, see the warnings about 'Documented arguments not in
\usage' in the r-devel checks.  These are from a recent bug fix
(PR#16223, see
<https://bugs.r-project.org/bugzilla/show_bug.cgi?id=16223>): can you
please fix your man pages as necessary?  (In most cases, remove the
documentation for argument '...'.)

Please correct before 2020-02-17 to safely retain your package on CRAN.

Best,
-k

@khufkens
Copy link
Member Author

khufkens commented Feb 3, 2020

@eliocamp, will fix next week.

@eliocamp
Copy link
Collaborator

eliocamp commented Feb 3, 2020

It's bizarre that the issue wasn't flagged in CRAN checks before 🧐️!

@khufkens
Copy link
Member Author

khufkens commented Feb 3, 2020

Yep, and still it only gives warnings in some occasions. Weird stuff.

@khufkens
Copy link
Member Author

khufkens commented Feb 4, 2020

Fixed as per:
7a9d920

@khufkens khufkens closed this as completed Feb 4, 2020
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

No branches or pull requests

2 participants