Skip to content

ci: defer draft builds and verify current pr state - #37

Merged
cloudartisan merged 3 commits into
mainfrom
ci/reduce-obsolete-actions-20261007
Oct 7, 2026
Merged

cloudartisan merged 3 commits into
mainfrom
ci/reduce-obsolete-actions-20261007

Conversation

@cloudartisan

@cloudartisan cloudartisan commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Drafts run cheap content/configuration checks, then explicitly fail full validation until currently ready. Live PR checks reject closed/draft PRs and stale head/base revisions before builds and before final success. Cancellation applies only to duplicate runs for the same workflow, PR, head and base; different revisions remain isolated. Non-PR verification uses unique keys. Existing tests, trigger/path filters, permissions, status names and deployment behavior are preserved.

Validation: 1,131 regression/event/preservation assertions, including out-of-order events and both demonstrated before-fix failures; actionlint syntax/expressions; pinned Hugo 0.160.1 development/production builds; 12 Python tests; 5,394 internal references with zero errors. Final head 0ec4e4e37bfcc0ef3f2d28022865172d5935a0d1 passed all four hosted checks and fresh Codex review with no findings: #37 (comment).

Merged as 6381ad83de88e97ffc3e9104b4a7f81904cb84e6 under David's explicit publishing approval. The existing Hugo publishing run https://github.com/cloudartisan/cloudartisan.github.io/actions/runs/37590150139 and Pages deployment https://github.com/cloudartisan/cloudartisan.github.io/actions/runs/37590204223 both succeeded. Published commit 5a7aeae1e0f32c4785a19a227eda3f8cae2e5651 records that exact source SHA. The live https://cloudartisan.com/ homepage returns HTTP 200 and matches the published index byte-for-byte. No account, repository, permission, protection or deployment settings changed.

Savings are from draft build deferral and duplicate-run cancellation. Cross-revision cancellation is deliberately reduced to protect current verification; monthly savings remain unmeasured.

@cloudartisan
cloudartisan marked this pull request as ready for review October 7, 2026 04:42
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-07T07:51:56.214032Z 0ec4e4e Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 73c7a35617

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/pr-validation.yml Outdated

Copy link
Copy Markdown
Owner Author

@codex review

@cloudartisan cloudartisan changed the title ci: defer draft builds and cancel obsolete pr validation ci: defer draft builds and verify current pr state Oct 7, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cbad2736bc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/pr-validation.yml Outdated

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Nice work!

Reviewed commit: 0ec4e4e37b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@cloudartisan
cloudartisan merged commit 6381ad8 into main Oct 7, 2026
4 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.

1 participant