Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[docs] CONTRIBUTING.md: specify bare infinitive instead of imperative#19064

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
wyattscarpenter wants to merge1 commit intopython:master
base:master
Choose a base branch
Loading
fromwyattscarpenter:patch-3

Conversation

wyattscarpenter
Copy link
Contributor

This PR changes the git commit guidelines to say "Use the bare infinitive in the subject line" instead of "Use the imperative mood in the subject line"

Technically, git commit subject lines are in the bare infinitive, not the imperative, because they're merely unconjugated, rather than being a command. (They are identical in form, though, so it's understandable that they were confused here.)

Or, so I think, at least! For more information, you can seehttps://en.wikipedia.org/wiki/Bare_infinitive ,https://en.wikipedia.org/wiki/Imperative_mood , or this blog post I wrote about this subject a while ago:https://wyattscarpenter.github.io/blog/git_commit_messages_are_in_the_bare_infinitive.txt

Technically, git commit subject lines are in the bare infinitive, not the imperative, because they're merely unconjugated, rather than being a command. (They are identical in form, though, so it's understandable that they were confused here.)Or, so I think, at least! For more information, you can seehttps://en.wikipedia.org/wiki/Bare_infinitive ,https://en.wikipedia.org/wiki/Imperative_mood , or this blog post I wrote about this subject a while ago:https://wyattscarpenter.github.io/blog/git_commit_messages_are_in_the_bare_infinitive.txt
@sterliakov
Copy link
Collaborator

To my surprise I was familiar with both terms and the proposed wording is correct, but... IMO this is overly pedantic. There are a lot of contributors here who only speak very basic English. The termimperative should be familiar to them as it's just basic grammar, whilebare infinitive may be less clear. I can easily imagine a commit/pr message "To prevent crash on a malformed input" because the author knows whatinfinitive means but never encounteredbare infinitive before and therefore just skipped it...

wyattscarpenter reacted with heart emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@wyattscarpenter@sterliakov

[8]ページ先頭

©2009-2025 Movatter.jp