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 commit message check #64

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

kgaikwad
Copy link
Collaborator

@kgaikwad kgaikwad commented Sep 8, 2023

No description provided.

pattern: '^(Refs|Fixes) #YUPTOO-[0-9]+ - .+$'
error: 'Commit message should start with "Fixes #YUPTOO-<issue number> - "'
pattern: '^(Refs|Fixes) #RHINENG-[0-9]+ - .+$'
error: 'Commit message should start with "Fixes #RHINENG-<issue number> - "'
Copy link
Contributor

@r14chandra r14chandra Sep 8, 2023

Choose a reason for hiding this comment

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

I just feel RHINENG-XXX should be fine as a prefix rather than adding Fixes or Refs #.
It is just my thought. Feel free to add your suggestions.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I will modify the pattern that will accept without these prefix Fixes/Refs as well.

@r14chandra
Copy link
Contributor

Your commit doesn't contain the JIRA card number. Tests are failing due to that.
Don't know if we should omit the check altogether for this type of scenario.

@codecov-commenter
Copy link

Codecov Report

Patch has no changes to coverable lines.

📢 Thoughts on this report? Let us know!.

@kgaikwad
Copy link
Collaborator Author

\retest

@kgaikwad kgaikwad merged commit 36d9066 into RedHatInsights:main Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants