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

Patch release of ggplot2 #4296

Closed
thomasp85 opened this issue Dec 14, 2020 · 6 comments
Closed

Patch release of ggplot2 #4296

thomasp85 opened this issue Dec 14, 2020 · 6 comments

Comments

@thomasp85
Copy link
Member

@hadley @clauswilke @yutannihilation @karawoo @paleolimbot

A slew of changes in R-devel/macOS has resulted in CRAN requesting a ggplot2 update (one of which is #4293). These updates are rather simple and superficial and the plan is to make a patch release off of the last release branch rather than go through the tedious process of a full release. I'll handle this and the master branch is still open for merging etc.

The patch release will be merged into master once on cran

@thomasp85 thomasp85 added this to the ggplot2 3.3.3 milestone Dec 14, 2020
@thomasp85
Copy link
Member Author

It seems everything is fixed on master already, so I'll just cherrypick those commits and do a patch release with these

@thomasp85
Copy link
Member Author

This release will fix the vdiffr and all.equal issues reported by CRAN. Further, it will include our transition to the MIT license and a whole bunch of link updating in the docs.

I'll submit today so we can get it out before the cran holidays (hopefully)

@yutannihilation
Copy link
Member

@thomasp85
Thanks for handling the release!
Sorry, I forgot I added a tweak that works only on GitHub Actions CI... This is my fault.

# If vdiffr is not available and visual tests are not explicitly disabled, raise error.
if (!identical(Sys.getenv("VDIFFR_RUN_TESTS"), "false")) {
abort("vdiffr is not installed")
}

Actually the CRAN check on M1 Mac is still failing.

  > test_check("ggplot2")
  Error: vdiffr is not installed
  Backtrace:
       █
    1. └─testthat::test_check("ggplot2")
    2.   └─testthat::test_dir(...)
    3.     └─testthat:::test_files(...)
    4.       └─testthat:::test_files(...)
    5.         └─testthat:::test_files_setup_state(...)
    6.           └─testthat::source_test_helpers(".", env)
    7.             └─testthat::source_dir(path, "^helper.*\\.[rR]$", env = env, wrap = FALSE)
    8.               └─base::lapply(files, source_file, env = env, chdir = chdir, wrap = wrap)
    9.                 └─testthat:::FUN(X[[i]], ...)
   10.                   └─base::eval(exprs, env)
   11.                     └─base::eval(exprs, env)
  Execution halted

(https://www.stats.ox.ac.uk/pub/bdr/M1mac/ggplot2.out)

@thomasp85
Copy link
Member Author

Goddammit. Let's see what happens. I'll make another patch if they contact me. I hope they can emphasise with the fact that we are shooting a bit in blind with M1 arc

@thomasp85
Copy link
Member Author

The first word was not directed at you btw🙂

@yutannihilation
Copy link
Member

I hope they can emphasise with the fact that we are shooting a bit in blind with M1 arc

Yeah, I hope so... In case they require another release merely for it, you can just pick #4310. I believe this should work on CRAN.

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

No branches or pull requests

2 participants