Skip to content

archive: preserve PR 1764 review output#1765

Merged
AceHack merged 3 commits into
mainfrom
claim/archive-pr-1764-review-substrate
May 7, 2026
Merged

archive: preserve PR 1764 review output#1765
AceHack merged 3 commits into
mainfrom
claim/archive-pr-1764-review-substrate

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 7, 2026

Summary

Checks

  • npx markdownlint-cli2 docs/history/pr-reviews/PR-1764-archive-preserve-pr-1763-review-output.md
  • git diff --check origin/main...HEAD

Notes

PR #1764 had no review threads; this routes the workflow-generated archive through PR review because direct pushes to main are guarded by repository rules.

Copilot AI review requested due to automatic review settings May 7, 2026 02:22
@AceHack AceHack enabled auto-merge (squash) May 7, 2026 02:22
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Preserves the post-merge, deterministic PR-review archive output for PR #1764 by committing the generated markdown archive and registering it in the PR archive manifest.

Changes:

  • Added a git-native PR-review archive markdown file for PR #1764 under docs/history/pr-reviews/.
  • Appended a new JSONL manifest entry for PR #1764 under docs/github/prs/manifest.jsonl.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/history/pr-reviews/PR-1764-archive-preserve-pr-1763-review-output.md Adds the generated PR #1764 review-archive markdown (metadata/outcome/threads).
docs/github/prs/manifest.jsonl Registers PR #1764’s archive path + metadata in the JSONL manifest.

Comment thread docs/github/prs/manifest.jsonl
@AceHack AceHack merged commit f3b0d19 into main May 7, 2026
25 checks passed
@AceHack AceHack deleted the claim/archive-pr-1764-review-substrate branch May 7, 2026 02:30
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