Skip to content

Use -F flag in git log in check-stable script #10283

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

Merged
merged 1 commit into from
Feb 21, 2020
Merged

Conversation

nebril
Copy link
Member

@nebril nebril commented Feb 21, 2020

If commit message is not a proper regular expression, check-stable
script crashes (bpf,option: Fix EPHE**R**MERAL typo is an example).

This change causes git log to not use commit message header as a
regex, but rather use simple string comparison.


This change is Reviewable

If commit message is not a proper regular expression, check-stable
script crashes (`bpf,option: Fix EPHE**R**MERAL typo` is an example).

This change causes `git log` to not use commit message header as a
regex, but rather use simple string comparison.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril requested a review from a team as a code owner February 21, 2020 10:42
@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

3 similar comments
@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@nebril nebril added the release-note/misc This PR makes changes that have no direct user impact. label Feb 21, 2020
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 45.533% when pulling f5ef72c on pr/fix-check-stable-crash into 0688f42 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants