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

Fix #1296, Adding commit-msg hook, copy to .git/hooks to enable. #1297

Closed
wants to merge 1 commit into from

Conversation

thesamprice
Copy link

Checklist (Please check before submitting)

Describe the contribution
Adds a githook that matches the regex that the server side checks for.

Testing performed
Steps taken to test the contribution:

  1. Build steps '...'
  2. Execution steps '...'

Expected behavior changes
A clear and concise description of how this contribution will change behavior and level of impact.

  • API Change: xxx (if applicable)
  • Behavior Change: xxx (if applicable)
  • Or no impact to behavior

System(s) tested on

  • Hardware: [e.g. PC, SP0, MCP750]
  • OS: [e.g. Ubuntu 18.04, RTEMS 4.11, VxWorks 6.9]
  • Versions: [e.g. cFE 6.6, OSAL 4.2, PSP 1.3 for mcp750, any related apps or tools]

Additional context
Add any other context about the contribution here.

Third party code
If included, identify any third party code and provide text file of license

Contributor Info - All information REQUIRED for consideration of pull request
Full name and company/organization/center of all contributors ("Personal" if individual work)

  • Note CLAs apply to only software contributions.

@thesamprice
Copy link
Author

Copy this to .git/hooks to activate this .githook.
Im using this to enforce the commit message format.

@dzbaker
Copy link
Collaborator

dzbaker commented Oct 6, 2022

@thesamprice CCB 6 October 2022: Best to configure on the developer's local git instance.

@dzbaker
Copy link
Collaborator

dzbaker commented Oct 27, 2022

CCB 27 October 2022: Agreed to close.

@dzbaker dzbaker closed this Oct 27, 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.

Enforce commit message name rule as githook
2 participants