-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
github/labeler: disable sync-labels #38158
Conversation
Yaml syntax cleaned too. Fixes: ceph#38107 (comment) Signed-off-by: Ernesto Puerta <epuertat@redhat.com>
9bbe700
to
59702b6
Compare
repo-token: "${{ secrets.GITHUB_TOKEN }}" | ||
- uses: actions/labeler@main | ||
with: | ||
sync-labels: '' |
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.
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.
@tchaikov after checking that code, my bet is that the defaults from the action.yml
are somehow injected in the environment, and that's why the empty string doesn't work:
> !!'false'
true
> !!''
false
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.
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.
Thanks @epuertat
I created this PR to see if we can get this fixed: actions/labeler#113 |
Yaml syntax cleaned too.
From wesnoth/wesnoth@958c82d
Fixes: #38107 (comment)
Signed-off-by: Ernesto Puerta epuertat@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox