-
Notifications
You must be signed in to change notification settings - Fork 3
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
Pulled from CRAN #30
Comments
Hi Steffan, I think this issue was that every now and then when the daily package tests ran on the CRAN server, the mtbls server would unreachable due to maintenance. I was thinking about moving the package on to bioconductor, but for now I'll change the examples/tests to skip so it should prevent this from happening. Will get it submitted back to CRAN this week. Thanks Tom |
It does not seem to be on CRAN. I am a bit nervous about testing directly against a live website. Would it be possible to scrape the responses from the website and then fake them for test purposes? That way you would have documentation of what the web used to do, way to see what it does now, CRAN would not be reaching across the world and tests would succeed reliably? |
Hey! I see this is still not available on CRAN. |
Hi @romanzenka , that's a great idea ! Does CRAN allow tests to return a warning ? |
I am thinking that this is not something CRAN should bother with - I would want my CRAN package to test robustly, without the vagaries of the internet, otherwise you are not testing your own work, but someone else's server. I would probably wrap that sort of test into some sort of internal function, and I'd run it on my own somewhere outside of CRAN, just to see how well is a given service performing over time. I would publish this package as: "assuming EBI works as it worked on |
Hi, CRAN says https://cran.r-project.org/web/packages/metabolighteR/index.html
What was the cause, and what'd be needed to get it back ?
Yours, Steffen
P.S.: I saw https://cran-archive.r-project.org/web/checks/2022/2022-11-21_check_results_metabolighteR.html
which says
and I am sure EBI still exists, so does not look like a road-blocker.
The text was updated successfully, but these errors were encountered: