-
Notifications
You must be signed in to change notification settings - Fork 156
Inclusive naming: the long game #655
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
Conversation
The pull request has 31 commits. The max allowed is 30. Please split the patch series into multiple pull requests. Also consider squashing related commits. |
The pull request has 34 commits. The max allowed is 30. Please split the patch series into multiple pull requests. Also consider squashing related commits. |
8c18f9a
to
28f007f
Compare
There is an issue in commit dbf118ff9974010a7d476609abfcae28223994c7: |
2e2f26f
to
b40fc5a
Compare
The pull request has 114 commits. The max allowed is 30. Please split the patch series into multiple pull requests. Also consider squashing related commits. |
The pull request has 115 commits. The max allowed is 30. Please split the patch series into multiple pull requests. Also consider squashing related commits. |
a293fd7
to
674ac49
Compare
071c381
to
456587b
Compare
|
Putting this down for the moment. Here is a list of files that have changed, but have not yet made it into a split-off PR:
|
Documentation/Makefile
Outdated
MAN_XML = $(patsubst %.txt,%.xml,$(MAN_TXT)) | ||
MAN_HTML = $(patsubst %.txt,%.html,$(MAN_TXT)) | ||
GIT_MAN_REF = master | ||
GIT_MAN_REF = main |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
I would recommend to split out one patch series at a time, when the previous patch series stabilized on the Git mailing list. Right now, for example, #656 saw its second iteration sent to the Git mailing list, there are still a couple of valid suggestions outstanding, so it is kind of stabilizing, but Junio has only picked up v1 as
|
TODO: should we touch this at all? Or is the branch name inherited from Perforce? Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
The part touched by this patch wants to remove uninteresting parts of commit messages, e.g. `Automatic merge of master`. Let's handle the branch name `main` as "uninteresting" in this context, because it has become the default branch name used by `git init`. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Some "mopping up" after the default branch name was changed. None of these should be user-facing. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This patch needs to be sent to the Git GUI project separately. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This patch needs to be sent to the `gitk` project individually. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This adjusts the `git cvsimport` command to reflect the new default branch name. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This adjusts the `git cvsimport` command to reflect the new default branch name. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This adjusts gitweb to reflect the new default branch name. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
8e6a6fb
to
36305c8
Compare
The pull request has 80 commits. The max allowed is 30. Please split the patch series into multiple pull requests. Also consider squashing related commits. |
There are merge commits in this Pull Request:
Please rebase the branch and force-push. |
d98b1dd
to
37b22c6
Compare
15d354c
to
377b9f9
Compare
f48423a
to
7e679a4
Compare
4ca4552
to
0c40f5c
Compare
This was a lot of work, yet — as time told — the core Git developers seem to be more interested in code style and real names in |
This PR is based on the initial idea of #653, but offers substantial improvements (amounting to a complete rewrite).
The idea is to split out several PRs from this:
Inclusive naming: the long game #655 (
init.defaultBranch
)Inclusive naming, part II #734 (
avoid-unnecessary-branch-name
)po: adjust for the recent change in send-pack/transport-helper #742 (
adjust-send-pack-translation
) (the i18n maintainer wants to do this differently).Avoid main as branch name in the test suite #743 (
avoid-main-as-branch-name-in-tests
)Prepare for changing the default branch name
main
(last manual part) #758 (use-main-as-default-branch-name-manual
)Adjust t5411 for the upcoming change of the default branch name #760 (
use-main-as-default-branch-name-t5411
)tests: consolidate the
file_size
function intotest-lib-functions.sh
#780 (file_size-test-function
)p4: respect init.defaultBranch #782 (
git-p4-and-init.defaultBranch
)(Mostly whitespace) cleanups in preparation for renaming the default main branch #784 (
main-branch-preliminary-cleanups
)Adjust t5515 for the upcoming change of the default branch name #761 (
use-main-as-default-branch-name-t5515
)tests: use
main
as default branch name #762 (use-main-as-default-branch-name
)t4013: prepare for the new default branch name "main" #783 (
prepare-for-main-branch-t4013
)use-main-as-default-branch-name-docs
use-main-as-default-branch-name-extra
git-gui/use-main-as-default-branch-name
gitk/use-main-as-default-branch-name
cvsimport-use-main-as-default-branch-name
p4-use-main-as-default-branch-name
gitweb-use-main-as-default-branch-name