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
[SPARK-34132][DOCS][R] Update Roxygen version references to 7.1.1 #31200
Conversation
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.
Please make a JIRA for this, @zero323 .
Done @dongjoon-hyun |
Test build #134123 has finished for PR 31200 at commit
|
Kubernetes integration test starting |
Kubernetes integration test status success |
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.
+1, LGTM. Thank you, @zero323 .
Merged to master/3.1.
### What changes were proposed in this pull request? This PR updates `roxygen2` version reference in docs and `DESCRIPTION` file. ### Why are the changes needed? According to information provided by shaneknapp (see [this comment](https://issues.apache.org/jira/browse/SPARK-30747?focusedCommentId=17265142&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17265142) to SPARK-30747) all workers use roxygen 7.1.1. In GitHub workflow we install the latest version https://github.com/apache/spark/blob/c75c29dcaa9458a9ce0dd7a4d5fafbffb4b7f6a6/.github/workflows/build_and_test.yml#L346 which [is also 7.1.1 at the moment](https://web.archive.org/web/20210115172522/https://cran.r-project.org/web/packages/roxygen2/). ### Does this PR introduce _any_ user-facing change? Docs and description mention currently used package verison. ### How was this patch tested? - `dev/lint-r`. - Manual check of command used in docs. Closes #31200 from zero323/ROXYGEN-VERSION-UPDATE-DOCS. Authored-by: zero323 <mszymkiewicz@gmail.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com> (cherry picked from commit 66cc129) Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
Thanks @dongjoon-hyun @HyukjinKwon @srowen |
What changes were proposed in this pull request?
This PR updates
roxygen2
version reference in docs andDESCRIPTION
file.Why are the changes needed?
According to information provided by @shaneknapp (see this comment to SPARK-30747) all workers use roxygen 7.1.1.
In GitHub workflow we install the latest version
spark/.github/workflows/build_and_test.yml
Line 346 in c75c29d
which is also 7.1.1 at the moment.
Does this PR introduce any user-facing change?
Docs and description mention currently used package verison.
How was this patch tested?
dev/lint-r
.