Skip to content

docs: record public release evidence - #4

Merged
Josh-Gi3r merged 1 commit into
mainfrom
docs/public-release-evidence
Aug 11, 2026
Merged

Josh-Gi3r merged 1 commit into
mainfrom
docs/public-release-evidence

Conversation

@Josh-Gi3r

Copy link
Copy Markdown
Owner

Summary

  • record public PR, merge, GitHub Actions, and clean-clone evidence
  • record protected main and the live desktop-v0.5.9 conflict-reporting path
  • remove pre-publication caveats that are no longer true

Validation

  • node scripts/audit-public-docs.mjs — 76 Markdown files, 143 local links
  • scripts/test-upstream-desktop-sync-contract.sh
  • git diff --check

This is an evidence-only follow-up to PRs #1 and #2.

Signed-off-by: Josh Gier <129348785+Josh-Gi3r@users.noreply.github.com>
@Josh-Gi3r
Josh-Gi3r merged commit c9b944e into main Aug 11, 2026
21 checks passed
@Josh-Gi3r
Josh-Gi3r deleted the docs/public-release-evidence branch August 11, 2026 13:02

@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: cb65f819b7

ℹ️ 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 FORK_PATCHES.md
Comment on lines +89 to +90
`d36f3933`. They were later published through reviewed PR #1; the resolver shell correction
was published through reviewed PR #2:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Add the required DCO sign-off

The reviewed commit message has no Signed-off-by trailer, so any PR containing this commit will fail the repository's required DCO Check. Recreate the commit with git commit -s before publishing it.

AGENTS.md reference: AGENTS.md:L112-L112

Useful? React with 👍 / 👎.

| Every crate denies all unsafe Rust | Unsupported | Platform-specific unsafe allowances exist. |
| The fork automatically installs upstream desktop releases | Unsupported | Branch automation detects stable release tags and prepares a reviewed PR; it never auto-merges. |
| Upstream release detection is currently live | Unverified | Its contract passed at local candidate `4e2e785d`, but the workflow still needs publication and an enabled successful default-branch run. |
| Upstream release detection is currently live | Verified for detection/conflict path | Default-branch run 31493736853 detected `desktop-v0.5.9`, aborted conflicts safely, and opened issue #3. Clean integration-PR creation remains unobserved. |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update the now-live upgrade guide

After this row classifies the detector as live for the detection/conflict path, the linked docs/development/fork-and-upgrades.md still calls it a “local candidate” and states that it has not been pushed, merged, or proven live. Readers following the evidence trail therefore receive mutually exclusive status information; update that guide to record the published workflow and observed conflict run as part of this evidence refresh.

Useful? React with 👍 / 👎.

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