Skip to content

Update CRAN comments and version for next release#96

Merged
karawoo merged 4 commits intor-lib:masterfrom
karawoo:prep-release
Aug 23, 2017
Merged

Update CRAN comments and version for next release#96
karawoo merged 4 commits intor-lib:masterfrom
karawoo:prep-release

Conversation

@karawoo
Copy link
Collaborator

@karawoo karawoo commented Aug 11, 2017

In prep for the next release of scales I've updated the version number and cran-comments.md. Is this the right information to include? This will be my first CRAN submission 😮 so I'm not sure what info they need.

Also I might've misinterpreted the instructions in our doc—I only built on Windows with R-devel, not R-release. Let me know if you want me to re-do that on both.

@karawoo karawoo requested a review from hadley August 11, 2017 20:44
DESCRIPTION Outdated
@@ -1,5 +1,5 @@
Package: scales
Version: 0.4.1.9002
Version: 0.5.1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be 0.5.0

cran-comments.md Outdated
## Reverse dependencies

This was minor release that only added new features, but I ran revdep checks on the 160 reverse dependencies just to be sure (complete results at https://github.com/hadley/scales/blob/master/revdep). I saw no problems related to scales.
I ran revdep checks on the 272 reverse dependencies (complete results at https://github.com/hadley/scales/blob/master/revdep). There were issues with three packages; one of these issues appears unrelated to `scales`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you inform the maintainers?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, for DiagrammeR and incidence.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In that case, I think you're probably providing a bit too much detail to CRAN. You can just say "the maintainers were informed of these issues on xxx xxxxx xxx". CRAN mainly wants to know that you've done due diligence.

DiagrammeR is fixed, so only one package has an issue now
@karawoo karawoo merged commit 21bf4c0 into r-lib:master Aug 23, 2017
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.

2 participants