Skip to content

Modernize GitHub Actions workflows#196

Merged
dennisoelkers merged 2 commits into
mainfrom
chore/modernize-workflows
Apr 24, 2026
Merged

Modernize GitHub Actions workflows#196
dennisoelkers merged 2 commits into
mainfrom
chore/modernize-workflows

Conversation

@dennisoelkers
Copy link
Copy Markdown
Member

Summary

  • Bump actions/checkout@v3 → v6 and actions/setup-node@v3 → v6 to clear the Node.js 20 deprecation warning (GitHub forces Node 24 on 2026-06-02 and removes Node 20 on 2026-09-16).
  • Upgrade the workflow Node.js runtime from 16.x (EOL since Sept 2023) to 22.x LTS.
  • Drop the duplicate push: tags trigger from release-package.yml. GitHub releases already push the tag, so both events fired on every publish and the second run always failed with "already published" (see run 24878697800 and its sibling 24878684711). Only release: published remains.

Note — does not fix the v4.0.1 publish failure

Run 24878697800 failed with Couldn't publish package: "https://registry.npmjs.org/@graylog%2fsawmill: Not found". That is the npm registry's response to a publish request carrying an invalid/expired token on a scoped package (404 is returned instead of 401/403 for privacy). The NPM_RELEASE_TOKEN org secret at Graylog2 was created 2023-11-28 and has never been rotated — it needs to be regenerated on npmjs.com and the org secret updated before a new v4.0.1 tag/release will publish.

Test plan

  • Merge and confirm the Build workflow runs green on a subsequent push/PR.
  • After rotating NPM_RELEASE_TOKEN, cut a patch release and confirm Publish Package to npmjs now runs exactly once (on release: published) and succeeds.

🤖 Generated with Claude Code

dennisoelkers and others added 2 commits April 24, 2026 11:13
Bump actions/checkout@v3 → v6 and actions/setup-node@v3 → v6 to clear
the Node.js 20 deprecation warning (forced to Node 24 on 2026-06-02).
Upgrade the workflow Node.js runtime from 16.x (EOL Sept 2023) to 22.x
LTS.

Drop the duplicate `push: tags` trigger from the release workflow. GitHub
releases already push the tag, so both events fired on every publish and
the second run always failed with "already published". Keep only the
`release: published` trigger, which is the deliberate action.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@dennisoelkers dennisoelkers merged commit e2c57e6 into main Apr 24, 2026
1 check passed
@dennisoelkers dennisoelkers deleted the chore/modernize-workflows branch April 24, 2026 09:38
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