Skip to content

Update GitHub Actions to use newer action versions and skip job status report on main branch.#270

Merged
nvaytet merged 4 commits intomainfrom
ci-action-update
Mar 26, 2026
Merged

Update GitHub Actions to use newer action versions and skip job status report on main branch.#270
nvaytet merged 4 commits intomainfrom
ci-action-update

Conversation

@YooSunYoung
Copy link
Copy Markdown
Member

I updated the actions that triggered Nodejs 20 deprecation warnings.

And then I realized that we don't need to run the job status report on the main branch so I added the condition.

@YooSunYoung YooSunYoung added the CI label Mar 26, 2026
@YooSunYoung
Copy link
Copy Markdown
Member Author

And can someone add the branch protection rule with CI / Report Job Status ??

Copy link
Copy Markdown
Member

@nvaytet nvaytet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder why the static analysis show up as 'required'... I didn't add it to the rule

Image

@nvaytet
Copy link
Copy Markdown
Member

nvaytet commented Mar 26, 2026

I cancelled the CI, there is no point burning more CI minutes on this.

@YooSunYoung
Copy link
Copy Markdown
Member Author

@nvaytet I just found out that the cancelled let the report job status pass???? Hmmmmm

@YooSunYoung
Copy link
Copy Markdown
Member Author

@nvaytet Can we make it optional to keep the branch up to date with the main branch? and enable auto-merge...?

@YooSunYoung
Copy link
Copy Markdown
Member Author

I cancelled the CI, there is no point burning more CI minutes on this.

I thought so too but I can't bypass the rule...
well... I wanted the freedom to be clumsy but it's kinda getting in our way...
Will you elevate my privilege...? 🌚

@nvaytet
Copy link
Copy Markdown
Member

nvaytet commented Mar 26, 2026

I'll just merge it ;-)

@nvaytet nvaytet merged commit f6b15b2 into main Mar 26, 2026
2 of 7 checks passed
@nvaytet nvaytet deleted the ci-action-update branch March 26, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants