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

Clarify commit message format in contributing guide #1643

Merged
merged 1 commit into from Nov 4, 2020
Merged

Clarify commit message format in contributing guide #1643

merged 1 commit into from Nov 4, 2020

Conversation

twpayne
Copy link
Contributor

@twpayne twpayne commented Nov 2, 2020

It took me a couple of tries to the get the commit message format right in #1642 as the CommitCheck check only states that a commit message is not valid without giving any explanation why, and the CommitCheck regular expression is not public.

This PR updates CONTRIBUTING.md to include a concrete example of what a commit message should look like.

RELEASE_NOTES=n/a

Signed-off-by: Tom Payne <twpayne@gmail.com>

More detailed description of commit, if needed.

RELEASE_NOTES=Description for release notes, or n/a if trivial.
Copy link
Member

Choose a reason for hiding this comment

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

Please list the typical tags we are using:
[DOCUMENTATION], [BUGFIX], [UX], [CLEANUP], [BREAKING], [DEPRECATION], [ENHANCEMENT], [FEATURE], [TESTING]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done in #1644.

Copy link
Member

@dominikschulz dominikschulz left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you! We should also add the tags, but not having that info available is very annoying, so merging as-is.

@dominikschulz dominikschulz merged commit 267f0ae into gopasspw:master Nov 4, 2020
@twpayne twpayne deleted the contributing branch November 4, 2020 20:13
kpitt pushed a commit to kpitt/gopass that referenced this pull request Jul 21, 2022
RELEASE_NOTES=n/a

Signed-off-by: Tom Payne <twpayne@gmail.com>
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

3 participants