-
Notifications
You must be signed in to change notification settings - Fork 11
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
noop-labels not working #49
Comments
@dza89 When you said it works as a string, do you mean that it works (and only works) when quoting the value? E.g. noop-labels:
- "documentation change" |
No, it works only as noop-labels: "documentation change" |
is it comma seperated eg. |
I can confirm the syntax is correct when passed in as a string as dza89 mentions. Unfortunately the "no-op" still does not work. As an example:
Adding the nobump label to the PR still results in the following error
|
For some additional context, this still does not work but it will at least be fed into the
|
I figured out what will probably help you solve this In my above example there are two tags specified for Tagging with either Tagging it with You likely want to manage this by using some other type of list and ensuring that you're iterating over that list, don't think that is happening currently and it's instead seeing the space delimited tags as a single tag. |
It's also worth noting that this will work
I believe this will also work
This WILL NOT work
|
Noob labels don't seem to work, config:
Results in a:
The workflow is not valid. .github/workflows/pull_request.yaml (Line: 110, Col: 21): A sequence was not expected
When I remove noop-labels it works fine.
The default in action.yaml is also an empty string, so something is up.
Edit: it works as a string.
The text was updated successfully, but these errors were encountered: