nflverse / nflfastR

A Set of Functions to Efficiently Scrape NFL Play by Play Data
https://www.nflfastr.com/
Other
414 stars 50 forks source link

Release nflfastR 4.6.1 #449

Closed mrcaseb closed 8 months ago

mrcaseb commented 8 months ago

CRAN forces us to submit a new version because they found this file https://github.com/nflverse/nflfastR/blob/master/tests/testthat/setup.R

It was intended to set data.table to two cores on CRAN machines but it never worked and we skip tests on cran.

So this release is as easy as delete above file and submit. CRAN submissions are closed until January 9th, so we have to wait.

If Uwe Ligges ever reads this: it's not our fault. You forced package developers with (reverse) dependencies on openMP to find a way around the multi threading problem and ultimately make everyone skip tests on CRAN. Bravo. Helps nobody.

Prepare for release:

Submit to CRAN:

Wait for CRAN...

mrcaseb commented 7 months ago

If we ever want to return to running tests on CRAN, we can try setting another environment variable as described here https://stat.ethz.ch/pipermail/r-package-devel/2023q4/009969.html