-
Notifications
You must be signed in to change notification settings - Fork 82
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
Replace http links with https links #201
Comments
@nicklucius Starting this as a separate issue because it's truly unique. I'm assuming that you want to call this |
@geneorama thanks, yep it will be 1.7.11-2. |
FYI, the URL for coveralls is not an HTTP/HTTPS issue. It looks like this is the original and replacement link pattern: Also, some HTTP links were not in the check, but thought this would be a good time to update them as well. I did the unnecessary HTTP changes in a separate commit so that they could be rolled back if there was a problem. Although I hesitate to make unnecessary changes, I felt that only updating the minimum necessary would fall more in the category of malicious compliance rather than scope creep. |
When I built I saw this message:
The @nicklucius do you know why it's not rebuilding the help? I'm concerned that the old HTTP links will remain in the documentation. |
This passed CRAN checks and is now live on CRAN. |
CRAN is requesting that we fix the links that are permanently moved, mostly to HTTPS
The relevant portion of the check results:
https://win-builder.r-project.org/incoming_pretest/RSocrata_1.7.11-1_20210911_035815/Debian/00check.log
https://win-builder.r-project.org/incoming_pretest/RSocrata_1.7.11-1_20210911_035815/Windows/00check.log
The text was updated successfully, but these errors were encountered: