Skip to content

.github/workflows: replace external set-commit-status action#45078

Merged
aanm merged 1 commit intomainfrom
pr/fix-commit-status-action
Mar 31, 2026
Merged

.github/workflows: replace external set-commit-status action#45078
aanm merged 1 commit intomainfrom
pr/fix-commit-status-action

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Mar 31, 2026

Replace myrotvorets/set-commit-status-action with an in-repo composite
action to prevent the Node.js 20 deprecation warning:
Warning: Node.js 20 actions are deprecated. The following actions are
running on Node.js 20 and may not work as expected:
myrotvorets/set-commit-status-action@3730c0a.
The set-commit-status action is simple enough (a single GitHub API call)
that we don't need to depend on an external dependency for it.

The tests have successfully passed in a test run https://github.com/cilium/cilium/actions/runs/23797204667

Until the PR is merged it's likely the tests will fail since the action doesn't exist on main.

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 31, 2026
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Mar 31, 2026

/test

@aanm aanm added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Mar 31, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 31, 2026
Replace myrotvorets/set-commit-status-action with an in-repo composite
action to prevent the Node.js 20 deprecation warning:
  Warning: Node.js 20 actions are deprecated. The following actions are
  running on Node.js 20 and may not work as expected:
  myrotvorets/set-commit-status-action@3730c0a.
The set-commit-status action is simple enough (a single GitHub API call)
that we don't need to depend on an external dependency for it.

Signed-off-by: André Martins <andre@cilium.io>
@cilium-ariane
Copy link
Copy Markdown

cilium-ariane bot commented Mar 31, 2026

/test

@aanm aanm force-pushed the pr/fix-commit-status-action branch from 2f628fc to b25f17e Compare March 31, 2026 13:10
@aanm aanm changed the title Pr/fix commit status action .github/workflows: replace external set-commit-status action Mar 31, 2026
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Mar 31, 2026

/test

@aanm aanm requested a review from Artyop March 31, 2026 13:11
@aanm aanm marked this pull request as ready for review March 31, 2026 13:12
@aanm aanm requested review from a team as code owners March 31, 2026 13:12
@aanm aanm requested review from ldelossa, pippolo84 and thorn3r March 31, 2026 13:12
Copy link
Copy Markdown
Contributor

@Artyop Artyop left a comment

Choose a reason for hiding this comment

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

using gh api instead of an external action using a typescript library? Nice 🚀

Copy link
Copy Markdown
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@aanm aanm merged commit a5c8739 into main Mar 31, 2026
103 of 108 checks passed
@aanm aanm deleted the pr/fix-commit-status-action branch March 31, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI Continuous Integration testing issue or flake needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants