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 guidelines and improve error messages #4184

Merged

Conversation

OhmSpectator
Copy link
Member

This PR includes the following changes:

Update commit message guidelines in CONTRIBUTING.md: Revised the commit message section to provide clearer instructions. Each commit message now includes a subject and a body, separated by an empty line. The body must start with a capital letter and include a "Signed-off-by" line at the end.

Clarify error message in check_commit_messages.py: Updated the script to provide more detailed error messages when a commit has no body. Added a link to the CONTRIBUTING.md file for more information on commit message guidelines.

Revised the commit message section to provide clearer instructions. Each
commit message now includes a subject and a body, separated by an empty
line. The body must start with a capital letter and include a
"Signed-off-by" line at the end.

Signed-off-by: Nikolay Martyanov <nikolay@zededa.com>
@github-actions github-actions bot requested a review from jsfakian August 29, 2024 17:36
@OhmSpectator OhmSpectator force-pushed the feature/improve-commit-linter-message branch from ff98361 to 35ee0f1 Compare August 29, 2024 17:39
Updated the script to provide more detailed error messages when a commit
has no body. Added a link to the CONTRIBUTING.md file for more
information on commit message guidelines.

Signed-off-by: Nikolay Martyanov <nikolay@zededa.com>
@OhmSpectator OhmSpectator force-pushed the feature/improve-commit-linter-message branch from 35ee0f1 to 9fe1462 Compare August 29, 2024 17:45
@OhmSpectator OhmSpectator self-assigned this Aug 29, 2024
@OhmSpectator
Copy link
Member Author

@rene, take a look, please =)

@OhmSpectator OhmSpectator requested a review from rene August 30, 2024 10:18
Copy link
Contributor

@rene rene left a comment

Choose a reason for hiding this comment

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

LGTM

@OhmSpectator OhmSpectator merged commit eafded3 into lf-edge:master Aug 30, 2024
47 of 53 checks passed
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.

3 participants