Skip to content

Bump workflow actions to current majors; retry flaky registry checks once - #41

Merged
Bunton33 merged 1 commit into
mainfrom
chore/actions-v7-and-validator-retry
Oct 25, 2026
Merged

Bunton33 merged 1 commit into
mainfrom
chore/actions-v7-and-validator-retry

Conversation

@Bunton33

Copy link
Copy Markdown
Contributor

What does this change?

Workflow actions. All six pinned actions move to their current major in one PR. This supersedes Dependabot's #35–#39. It also covers the upload-pages-artifact bump that Dependabot couldn't open (its five-open-PR limit):

Action From To
actions/checkout v4.4.0 v7.0.1
actions/setup-python v5.6.0 v7.0.0
actions/setup-node v4.4.0 v7.1.0
actions/upload-artifact v4.6.2 v7.0.2
actions/upload-pages-artifact v3.0.1 v5.0.0
actions/deploy-pages v4.0.5 v5.0.1

Dependabot now groups all action bumps into one PR, so future updates merge and deploy together.

Validator retry. Each network request is retried once after 5s when the failure may be transient (connection error, timeout, 5xx). A 4xx is not retried. #40 was the nightly failing on a single failed load of AIR-1's classic installer page: no deploy happened there, and the next two nightlies passed. The registry job gets timeout-minutes: 20, so a total upstream outage fails fast.

For reviewers

  • Deploy actions: upload-pages-artifact and deploy-pages only run in pages.yml, on main. The github-pages environment only accepts main, so this PR's CI can't exercise them. Their first run is the deploy after merge. If it fails, the current site stays live.
  • upload-artifact only runs when a UI test fails, so green CI doesn't exercise it either.
  • setup-python: I kept it at v7.0.0, the version Dependabot proposed. v7.1.0, which only adds features, came out three days ago; the grouped Dependabot update will pick it up.
  • Review: two independent reviewers checked the SHAs, the inputs used across each major jump, and the retry behavior. They found no defects. Their two test gaps are covered: a HEAD never reads a body, and relative manifests resolve against the redirected URL.

Fixes #40. Once merged, Dependabot closes #35–#39 as superseded.

Checklist

  • Playwright tests pass locally (cd tests && npx playwright test): no site files changed; CI runs them
  • Any new runtime file/dir is added to the cp list in .github/workflows/pages.yml: none added
  • No new build step, framework, or runtime CDN dependency
  • If devices.json changed: unchanged. python scripts/validate_registry.py passes (full and --offline), and 79 + 14 Python unit tests pass

🤖 Generated with Claude Code

Moves every pinned action to its current major in one change: checkout
v7.0.1, setup-python v7.0.0, setup-node v7.1.0, upload-artifact v7.0.2,
upload-pages-artifact v5.0.0 and deploy-pages v5.0.1 (all Node 24; no input
we use changed). Dependabot now groups action bumps into a single PR, so
they merge and deploy together instead of queueing behind its five-PR limit.

The validator retries a request once, after 5s, when it fails in a way that
may be transient (connection error, timeout, 5xx); a 4xx is not retried.
The 2026-10-23 nightly opened issue #40 because AIR-1's classic installer
page failed to load once. The registry job gets a 20-minute timeout so a
total upstream outage fails fast instead of running for hours.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 25, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f8f38c9c-b1de-4b39-9b2f-2f8f811322a1
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Bunton33
Bunton33 merged commit a78ebe1 into main Oct 25, 2026
3 checks passed
@Bunton33
Bunton33 deleted the chore/actions-v7-and-validator-retry branch October 25, 2026 17:37
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.

Nightly registry check failed

1 participant