-
Notifications
You must be signed in to change notification settings - Fork 5
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
No lowercasing of branch to fix pipeline matching #714
Conversation
LGTM. I would add a note to the changelog as this is a user-facing change. It fixes bug #713. |
Makes sense will do that soon. |
CHANGELOG.md
Outdated
@@ -20,6 +20,8 @@ listed in the changelog. | |||
|
|||
- Gradle build dir is now configurable ([#703](https://github.com/opendevstack/ods-pipeline/pull/703)) | |||
|
|||
- ods.yaml branch trigger patterns must be lowercase ([#713](https://github.com/opendevstack/ods-pipeline/issues/713)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is an outdated changelog. If you rebase on master you'll see that there is a 0.13.2 release and the line you added should actually go above that, into the "Unreleased" section. I would suggest under "Fixed" - or "Changed"?
eb17845
to
5c23cb9
Compare
Hmm, the Go task tests fail in the sonar step. Do you have an idea if / how that could be related? |
Actually, https://github.com/opendevstack/ods-pipeline/actions/runs/5584494350 failed as well, also in regards to SQ, but for different tasks. Maybe there is a general issue I need to look into. |
:-) I was starting to look at it. But haven't found an issue. There should be no impact on the go tasks though in this PR I'd say. |
Fixes #713
Tasks:
docs/design
directory or not applicabledocs
directory or not applicablemake test
) or not applicable