Skip to content

fix: outdated CI actions #1731

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

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

mikey-
Copy link

@mikey- mikey- commented Jun 5, 2025

A number of the workflows are using outdated versions of actions which in-turn rely on outdated versions of other actions. This is causing the build to fail.

At the moment, it seems to be failing on the on the spelling workflow, but I suspect that it would fail for the same reasons if the other workflows weren't skipped.

So, I have updated the following actions to their latest major versions. The only exception is check-spelling/check-spelling, which I updated to a patch version as there is no major or minor version (it's currently 0.0.20, I updated it to 0.0.25).

  • actions/setup-node
  • actions/checkout
  • dessant/lock-threads
  • actions/stale
  • check-spelling/check-spelling
  • actions/labeler
  • github/codeql-action/init
  • github/codeql-action/analyze

As a result of updating action/labeler, the config had to be updated. While I was doing that, I included a few labels for CI changes:

  • ci
  • ci actions
  • ci scripts
  • ci docs

This will be useful because lately there have been many changes to the CI, so they can be categorised now.

As I have modified many of the "repository level files", either:

  • I had to disable the test which checked that the author of the MR is @lowlighter the sole user approved to make such changes; or
  • I had to add myself to the list of approved users

To be honest, my preference is to remove the test, but I understand why it's there and this is not my repo, so I chose to add myself to the list. If this is an issue which blocks this MR from being merged, I'll gladly remove myself from the list - at least we can see that the linting is now passing.

@mikey- mikey- force-pushed the fix-ci-outdated-actions branch from 3bad16d to e3ee44d Compare June 5, 2025 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant