-
Notifications
You must be signed in to change notification settings - Fork 810
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
Bump GitHub workflow actions to latest versions #3355
Bump GitHub workflow actions to latest versions #3355
Conversation
Build Succeeded 👏 Build Id: e1775258-3325-4938-9525-9c0b2cef5854 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Looks good. /lgtm |
@Kalaiselvi84 you've worked the most on github actions, just wanted to triple check with you before approving. You see any issues? |
I will test in my fork and update here shortly.. |
@markmandel, I tested the Label-PR script with the elevated version and it's not working as expected. Here's the link |
Sorry, can you point out the issue, I'm having a hard time seeing it. |
I received the following error: Error log: https://gist.github.com/Kalaiselvi84/bdc04f75d76008fc17ded1422cdd88f1 If I understand correctly, I've tried using May I know how are we testing the other Action workflows(close, stale, and obsolete)? |
Interesting, is this because there was no label specified in the description? (Although I look at the logs, it seems like there is?) If we can't upgrade, then we can't upgrade -- warnings are not the end of the world. or flip side @deining if you are keen to update the script so it will work, that would be good too.
Honestly, probably just making the change and seeing what happens (which is less than ideal, so we may not want to change that one). |
Yes, I chose https://screenshot.googleplex.com/5sRr2AwNauSM2w7
I agree with your suggestion👍🏻 |
Either way if |
https://github.com/actions/github-script#apply-a-label-to-an-issue It looks like the github-script has been tweaked a bit and should be reformatted as follows github.rest.issues.addLabels
|
@aimuz, Thanks for sharing |
@deining are you able to make those updates to this PR (and also resolve the conflicts please)? |
Build Failed 😱 Build Id: c2cf2482-c018-498a-ba36-3c82175ce50a To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
@deining, would you mind appending a line to the end of the obsolete, stale, and close.yaml files to address the linting issue, please? |
Done. I also rebase my branch, it is not out-of-date any more. |
06d3695
to
9482df2
Compare
Build Failed 😱 Build Id: 41bf4813-19c4-4824-bcc8-33d6cc490bec To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
9482df2
to
2d12f4a
Compare
Build Failed 😱 Build Id: 08a3d078-275e-4fbd-b7a6-46a4e13c1bbc To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
2d12f4a
to
e9bed21
Compare
e9bed21
to
2f0b020
Compare
Build Succeeded 👏 Build Id: e68928c8-fd48-469e-9d87-6bd717a411d7 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
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
@Kalaiselvi84 you have the most experience with these tools - any objection to approving and merging? |
LGTM! |
HERE WE GO 😃 |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aimuz, deining, markmandel The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / Why we need it:
This PR bumps GitHub workflow actions to their latest versions. This avoids warning messages as seen here.