-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
GH-44570: [Release][R][Docs] Update r/pkgdown/assets/versions.html
#44572
Conversation
|
aa004d7
to
34d5442
Compare
34d5442
to
999c1a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this
@@ -194,6 +194,7 @@ update_versions() { | |||
"${base_version}" \ | |||
"${next_version}" | |||
git add docs/source/_static/versions.json | |||
git add r/pkgdown/assets/versions.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aaah was this missing so we weren't actually updating that page? Oops!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. :-)
After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 191472d. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Rationale for this change
It's still used at https://arrow.apache.org/docs/r/versions.html . So we should update it too.
What changes are included in this PR?
Commit it.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.
r/pkgdown/assets/versions.html
isn't updated #44570