Skip to content

ci: bump deprecated actions; replace archived release action#234

Merged
cameroncuster merged 1 commit into
devfrom
ci-actions-bump
Jul 8, 2026
Merged

ci: bump deprecated actions; replace archived release action#234
cameroncuster merged 1 commit into
devfrom
ci-actions-bump

Conversation

@cameroncuster

Copy link
Copy Markdown
Member

Replaces deprecated/unmaintained GitHub Actions:

  • actions/checkout@v1@v4 and actions/setup-python@v1@v5 in library_checker_aizu (v1 targets Node versions that runners are deprecating; the rest of the workflow already uses v4)
  • marvinpinto/action-automatic-releases@latestsoftprops/action-gh-release@v2 in publish_pdf (marvinpinto's action is archived and unmaintained; pinned @latest is also a supply-chain risk)

Details on the release step:

  • softprops/action-gh-release@v2 uses the default GITHUB_TOKEN; the workflow already grants permissions: contents: write
  • tag_name: ptc / name: ptc reproduces the existing rolling-release behavior (updates the same ptc release/tag in place)
  • download-artifact path changed from tests/ptc.pdf to tests/: with download-artifact@v4, path is the destination directory, so the old value produced tests/ptc.pdf/ptc.pdf; now tests/ptc.pdf is the file itself

Release behavior should be verified on the first dev push after merge (release ptc gets a fresh ptc.pdf asset).

@cameroncuster cameroncuster merged commit 78f67d3 into dev Jul 8, 2026
8 checks passed
@cameroncuster cameroncuster deleted the ci-actions-bump branch July 8, 2026 04:16
cameroncuster added a commit that referenced this pull request Jul 9, 2026
actions/checkout@v4 defaults to a shallow clone (fetch-depth: 1), which
broke oj-verify's timestamp-based change detection (regressed in #234, when
checkout went v1 -> v4). Pin fetch-depth: 0 so only changed tests re-run.
cameroncuster added a commit that referenced this pull request Jul 9, 2026
actions/checkout@v4 defaults to a shallow clone (fetch-depth: 1), which
broke oj-verify's timestamp-based change detection (regressed in #234, when
checkout went v1 -> v4). Pin fetch-depth: 0 so only changed tests re-run.
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