-
Notifications
You must be signed in to change notification settings - Fork 73
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 re-submission #1044
Comments
This seems like a needle in the heystack thing to find -.-
|
I am trying to reproduce the problem with {rhub}:
|
@krlmlr I guess here we might need your help. |
Do I understand correctly that none of these checks have shown the error? Is there a way to find out when this error started to occur? A hint in the most recent R-devel changes? Unfortunately, the Wayback Machine is of no help here, they don't archive CRAN check results. |
IIRC, yes, I did. But there was only the Is the problem that we are leaving the cache directory behind, or something similar? Btw, we error on a styler/.github/workflows/check-full.yaml Line 59 in 4c1cb51
|
yes.
yes, tried that also. |
I don’t think so, these problems looked differently: #836 |
These machines are fairly special:
Maybe we can even replicate the problems locally? |
Mirracles happen. r-devel-linux-x86_64-debian-gcc no longer shows a note. 😮. Maybe it was just a bug in R devel. Let's see if the other note also disappears. |
@krlmlr do you think we should to write on r devel? I have never done that before. 7 days till removal. |
Let's wait for five more days, and then push an update to CRAN if the failures persist. This will buy us a bit more time. |
Ok, and what update should we push if we don't even know what the root cause is? |
We could disable all tests to see if problems still occur. Then, we at
least know if it's the tests or the examples.
Another thing to try would be to run R CMD check on Linux devel with the
flags described in
https://www.stats.ox.ac.uk/pub/bdr/Rconfig/r-devel-linux-x86_64-fedora-gcc
. It puzzles me that _R_CHECK_THINGS_IN_TEMP_DIR_ is set to "true" both
here and in the other custom Fedora setting, but the other Fedora
setting now works. I really hope this is a transient thing.
|
Also, the last checks with this R version ran with
Let's give it some more time. |
For your last point: Agree, all other checks have newer R versions. |
@krlmlr days passed and the NOTE still appears on one machine, which uses the same old 20222-10-21 R devel. Should we re-submit with all tests skipped? How would I do that? Just comment out the code in |
Commenting out sounds good. The error is on CRAN's end, but we don't want styler to be pulled, which is likely to happen automatically if we don't act. And acting means resubmitting, e-mails are likely to be ignored. |
Update is on CRAN! 🎉 |
Dear maintainer,
Please see the problems shown on
https://cran.r-project.org/web/checks/check_results_styler.html.
Please correct before 2022-11-09 to safely retain your package on CRAN.
The CRAN Team
Seems like only occurring on R devel with fedora flavor. Also, it's not clear if the problem was already present for version 1.7.0, as the below screenshot has no data for 1.7.0 anymore. @IndrajeetPatil did you check the CRAN results page before 1.8.0 wsa released?
The text was updated successfully, but these errors were encountered: