-
Notifications
You must be signed in to change notification settings - Fork 48
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
🌱 Update deprecated pr labeler version #276
Conversation
Bump the version of the PR Labeler action, updating to the newer configuration schema. The size configuration was changed a few versions ago. I'm looking to remove support so I'm sending some PRs to the very few repos that still rely on it. I updated the reference to the action to master. This actually uses the latest stable version so it should avoid any unintended breakage. Thanks for using the action! Signed-off-by: Galo Navarro <[email protected]>
@srvaroa, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@srvaroa, VMware has approved your signed contributor license agreement. |
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.
LGTM
.github/workflows/pr-labeler.yml
Outdated
@@ -31,7 +31,7 @@ jobs: | |||
# - "config/crd/bases/.*" # generated CRDs | |||
# - "docs/apis/v*.md" # generated API documentation, | |||
# # ex. docs/apis/v1alpha1.md | |||
- uses: srvaroa/labeler@v0.9 | |||
- uses: srvaroa/labeler@master |
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.
We want to avoid sliding version targets. Please point to a specific version instead.
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 for the review, fixed!
@srvaroa By the way, thank you very much for such a fantastic GitHub action! |
Pin to v1.9.0
Thanks @akutz ! I have pinned the version to the latest available. |
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.
Thank you!
Minimum allowed line rate is |
What does this PR do, and why is it needed?
Bump the version of the PR Labeler action, updating to the newer configuration schema. The size configuration was changed a few versions ago. I'm looking to remove support so I'm sending some PRs to the very few repos that still rely on it.
I updated the reference to the action to master. This actually uses the latest stable version so it should avoid any unintended breakage.
Thanks for using the action!