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

Minor fixes #269

Merged
merged 3 commits into from
Oct 14, 2024
Merged

Minor fixes #269

merged 3 commits into from
Oct 14, 2024

Conversation

pbulsink
Copy link
Collaborator

Updated in plot_fastest to remove a unchecked reliance on Ergast. This was the cause of some of the routine chk action failures.

Tried to fix the downloads badge.

Note that other Ergast server issues will cause this to fail but the affected lines of code will pass their tests

@pbulsink pbulsink requested a review from SCasanova September 26, 2024 00:12
@pbulsink pbulsink linked an issue Sep 26, 2024 that may be closed by this pull request
@pbulsink pbulsink mentioned this pull request Oct 12, 2024
@SCasanova
Copy link
Owner

Should 270 or 269 be merged first?

@pbulsink
Copy link
Collaborator Author

Best to do #269, then rebase #270 and make sure there's no conflicts. I can do that tonight

@SCasanova SCasanova merged commit ed87918 into SCasanova:master Oct 14, 2024
2 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CRAN downloads not showing properly on README
2 participants