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

Harmonize commitish and committish in docstrings #799

Merged
merged 1 commit into from Feb 7, 2022

Conversation

sgasse
Copy link
Contributor

@sgasse sgasse commented Feb 5, 2022

The git glossary mentions both verions. However in-code, only the first
variant is used. This commit harmonizes the usages:
https://git-scm.com/docs/gitglossary#Documentation/
gitglossary.txt-aiddefcommit-ishacommit-ishalsocommittish

The git glossary mentions both verions. However in-code, only the first
variant is used. This commit harmonizes the usages:
https://git-scm.com/docs/gitglossary#Documentation/
gitglossary.txt-aiddefcommit-ishacommit-ishalsocommittish
Comment on lines -834 to 836
/// If the provided committish cannot be found in the repository, the HEAD
/// If the provided commitish cannot be found in the repository, the HEAD
/// is unaltered and an error is returned.
/// If the provided commitish cannot be peeled into a commit, the HEAD is
Copy link
Contributor Author

Choose a reason for hiding this comment

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

e.g. here we have both spellings directly one after another - so maybe it makes sense to harmonize it? Or does it not make any sense because it we might get both variants back anyway?

@alexcrichton
Copy link
Member

Seems reasonable!

@alexcrichton alexcrichton merged commit ae02afd into rust-lang:master Feb 7, 2022
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.

None yet

2 participants