Skip to content

ci: 👷 add deploy preview and cleanup jobs to deploy workflow - #24

Closed
iamnewton wants to merge 7 commits into
mainfrom
ci/add-deploy-preview
Closed

ci: 👷 add deploy preview and cleanup jobs to deploy workflow#24
iamnewton wants to merge 7 commits into
mainfrom
ci/add-deploy-preview

Conversation

@iamnewton

Copy link
Copy Markdown
Contributor

Adds deploy preview and cleanup to the deploy workflow, matching the setup already in theholocron/clients and theholocron/holocron.

deploy.yml:

  • Add pull_request trigger with types: [opened, synchronize, reopened, closed] and matching paths
  • Update concurrency to use a PR-scoped group and skip cancellation on closed events
  • Expand permissions to include deployments: write and pull-requests: write
  • Guard deploy job with if: github.event_name != 'pull_request'
  • Add preview job (Cloudflare Pages branch deployment on PR open/update)
  • Add cleanup job (deletes the CF Pages deployment on PR close/merge)

holocron.config.ts:

  • Add preview: { project: "theholocron-preview", domain: "preview.theholocron.dev" } to the deploy workflow config so holocron sync regenerates this correctly in the future.

Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>
@iamnewton
iamnewton requested a review from a team as a code owner August 28, 2026 03:30
@github-actions github-actions Bot added the ci CI/CD pipeline changes label Aug 28, 2026
Signed-off-by: super-linterbot[bot] <310376783+super-linterbot[bot]@users.noreply.github.com>
@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #24   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines          105       105           
  Branches        18        18           
=========================================
  Hits           105       105           
Components Coverage Δ
components-doc 100.00% <ø> (ø)
registry-doc 100.00% <ø> (ø)

…: true

Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>
domain field on HolocronConfig was added after 3.29.12; bumping to the
current release unblocks the tsc check in the deploy preview PR.

Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>
domain field on HolocronConfig was added after 3.29.12; bumping to the
current release unblocks the tsc check.

Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>
@github-actions
github-actions Bot temporarily deployed to theholocron-preview (Preview) August 28, 2026 19:38 Inactive
@iamnewton

Copy link
Copy Markdown
Contributor Author

Superseded by #29 — the nodeDocs() preset adoption already includes deploy+preview.

@iamnewton iamnewton closed this Aug 28, 2026
@iamnewton
iamnewton deleted the ci/add-deploy-preview branch August 28, 2026 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/CD pipeline changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant