Upgrade to roxygen2 7.0.1 #3637
Conversation
Including converting all uses of `\%` back to `%`, and removing roxygen2 tags that the new warnings indicated were superfluous.
|
Hmmm that's a little concerning. @thomasp85 can you please take a look? 3 unit tests are failing but only for R-release on linux |
|
All the failing tests are visual ones. I think we can just update the visual cases, but I don't know well about the details of the latest release of scales package, so could you confirm this is expected? I created another PR to fix this: #3638 In any case, I bet updating roxygen2 is irrelevant to the failures. |
|
The scales package changed the code used to calculate color ramps (see here: r-lib/scales@eb24db7). As a result, I would expect all visual tests with color ramps to fail. I would recommend visually checking that the new color ramps look fine and then accepting the new test cases. |
|
Oh, thanks for identifying the commit.
Yes, I've checked on my local. I'll post the comparison on #3638. |
|
Thanks all! I've merged the latest master into this branch so hopefully the next round of checks will pass. |
Including converting all uses of
\%back to%, and removing roxygen2 tags that the new warnings indicated were superfluous.