layer5 icon indicating copy to clipboard operation
layer5 copied to clipboard

[CI] Update actions version and remove warnings

Open Nitish-bot opened this issue 7 months ago • 3 comments

Description The current versioning was very old making us susceptible to security vulnerabilities and to using deprecated methods that currently throw warnings wherever and old version actions/checkout is used

This PR fixes #

Notes for Reviewers

  • Versioning should absolutely not refer to main or master branch since these can include breaking changes.
  • Versioning can either reference a fixed version (e.g., v4.2.2) or a flexible one (e.g., v4, which resolves to the latest v4.x.x). In this PR, I’ve opted to pin the version, but we can switch to a more flexible reference.

Signed commits

  • [x] Yes, I signed my commits.

Nitish-bot avatar Jun 13 '25 07:06 Nitish-bot

🚀 Preview for commit 58911110dda395121158127769b818e2934e1850 at: https://684bd9c4a8dcba4774e46afa--layer5.netlify.app

l5io avatar Jun 13 '25 07:06 l5io

🚀 Preview for commit df3e93fd6489aaf6ea26956f54c2971f168dd8e9 at: https://684c03e76c20e0007d3e57c2--layer5.netlify.app

l5io avatar Jun 13 '25 10:06 l5io

There is an argument to be made here, 'don't fix what ain't broke'. I differ in opinion and would say for the minimal amount of effort that this takes (which would be fixing any anomalies that show up as a result of these changes), it's well worth it to use newer versions.

My reasoning for not pointing to master or main branches of actions can be found here

Nitish-bot avatar Jun 14 '25 08:06 Nitish-bot

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 27 '25 00:06 stale[bot]