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

GH-20161: [Website] Remove mentions of master branch from Apache Arrow website content #324

Merged
merged 7 commits into from Mar 1, 2023

Conversation

kevingurney
Copy link
Member

@kevingurney kevingurney commented Feb 28, 2023

Overview

As part of apache/arrow#31142 and in response to the recent rename of the apache/arrow-site repository default branch to main, this pull request removes mentions of the "master" branch from the Apache Arrow website content.

Qualification

To qualify these changes, I:

  1. Performed a local build and preview of the website using bundle exec rake and manually verified that all the links are still working.

Future Directions

  1. There are still some remaining tasks listed in [Dev] Rename default branch to 'main' arrow#31142 related to the renaming of the default branch of the apache/arrow repo (and related repositories) to main.

Notes

Note: I left the mentions of "master" in the _release and _posts directory unmodified. As described in apache/arrow#20161 (comment), it's unclear whether it makes sense to change these uses since these are referring to a past state of the repository. If the community would prefer that we also removes mentions of "master" from these directories, I would be more than happy to do so.

Closes apache/arrow#20161.

@kevingurney
Copy link
Member Author

Note: The CI failures in the Build step are unrelated to this change. This is a result of #325, which is being addressed separately.

@kevingurney
Copy link
Member Author

@jorisvandenbossche, if you have the bandwidth, would you be willing to review this PR, as well?

If not, would you mind pointing me towards someone else in the community who might be interested in reviewing these changes?

Thank you!

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit a63941c into apache:main Mar 1, 2023
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.

[Website] Remove mentions of master branch from Apache Arrow website content
2 participants