-
Notifications
You must be signed in to change notification settings - Fork 3.9k
MINOR: [Python] Clean up unused link references in README #48249
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: [Python] Clean up unused link references in README #48249
Conversation
|
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format? or See also: |
The python/README.md previously pointed to developers/python.html. Updated to the current developers/python/index.html location. Signed-off-by: Julien Stephan <jstephan@baylibre.com>
d8fce71 to
1e5c174
Compare
raulcd
left a comment
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 the fix!
|
I just added a new commit here instead of opening a new PR, since I realized that there are 2 links in the same files that are defined but not used and should be removed |
Links [3] and [5] in python/README.md are no longer referenced and can be safely removed. Additionally, renumber the remaining link definitions so that link [6] becomes [3]. Signed-off-by: Julien Stephan <jstephan@baylibre.com>
042ccb5 to
d3f577d
Compare
|
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit b6a1375. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Rationale for this change
The link to the python development documentation in python/README.md is broken: https://arrow.apache.org/docs/developers/python.html
What changes are included in this PR?
This PR fix the link to point to the new location: https://arrow.apache.org/docs/developers/python/index.html
Also removes unused links
Are these changes tested?
Yes, link is now functional
Are there any user-facing changes?
No