Skip to content

ci: publish gated beta and next candidates - #132

Merged
rdlabo merged 5 commits into
mainfrom
feat/pr-beta-publish
Aug 29, 2026
Merged

ci: publish gated beta and next candidates#132
rdlabo merged 5 commits into
mainfrom
feat/pr-beta-publish

Conversation

@rdlabo

@rdlabo rdlabo commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Build immutable npm candidates in a read-only workflow with no publishing credentials.
  • Publish an open PR head to the beta dist-tag only after its required CI passes and a repository administrator comments /beta.
  • Publish the current main head to the next dist-tag only after all required main CI passes for that exact commit.
  • Comment the immutable version and exact install command on the associated pull request.
  • Keep tag-based prereleases on next.

Security and release invariants

  • The privileged Release workflow never checks out or executes pull request code.
  • /beta authorizes one exact PR head SHA. A new commit requires new CI and a fresh administrator /beta comment.
  • The required workflow names and paths, successful source run, immutable artifact, package name, version, and npm registry are verified before publishing.
  • A PR that modifies a release-gating workflow cannot be beta-published until that workflow change is on main.
  • Candidate lifecycle scripts are disabled during packing and publishing.
  • Candidate publishes explicitly use only beta or next. They cannot update latest.
  • Only an explicit stable vX.Y.Z tag uses npm's default latest dist-tag; prerelease version tags use next.

Rollout test

After this PR lands on main, merge main into PR #131 (fix/radio-group-across-lists) without merging that feature PR. Let its CI pass, then test the administrator /beta flow and exact install comment there.

Validation

  • actionlint 1.7.12 passed for both changed workflows.
  • Prettier and git diff --check passed.
  • CodeQL Actions and JavaScript analyses passed with no open code-scanning alerts on the current head.
  • Lint, Package Candidate, and Ionic 8/9 E2E checks passed.
  • The generated artifact was inspected as @rdlabo/ionic-theme-ios26@9.0.2-beta.pr132.shaa6fb35b1bd00.

@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Playwright test results

passed  90 passed

Details

stats  90 tests across 2 suites
duration  1 minute, 54 seconds
commit  a6fb35b
info  This detailed result covers Ionic 9 only. Ionic 8 runs against the same screenshots in a separate matrix job; check the workflow run for both results. To update the screenshots, comment with /update-screenshots.

@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-ios26/pr-132/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

github-actions Bot added a commit that referenced this pull request Aug 29, 2026
Comment thread .github/workflows/release.yml Fixed
Comment thread .github/workflows/release.yml Fixed
Comment thread .github/workflows/release.yml Fixed
Comment thread .github/workflows/release.yml Fixed
Comment thread .github/workflows/release.yml Fixed
@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-ios26/pr-132/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

github-actions Bot added a commit that referenced this pull request Aug 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-ios26/pr-132/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

github-actions Bot added a commit that referenced this pull request Aug 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-ios26/pr-132/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

github-actions Bot added a commit that referenced this pull request Aug 29, 2026
@rdlabo rdlabo changed the title ci: publish PR betas after admin approval ci: publish gated beta and next candidates Aug 29, 2026
@rdlabo
rdlabo merged commit 0c6bf33 into main Aug 29, 2026
11 checks passed
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.

2 participants