Skip to content

docs(changelog): correct drifted PR refs and fill 4.0.2 gaps#2816

Merged
bpamiri merged 1 commit into
developfrom
docs/changelog-unreleased-4-0-2
May 27, 2026
Merged

docs(changelog): correct drifted PR refs and fill 4.0.2 gaps#2816
bpamiri merged 1 commit into
developfrom
docs/changelog-unreleased-4-0-2

Conversation

@bpamiri

@bpamiri bpamiri commented May 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

The [Unreleased] section already held most of the 4.0.2 changeset, but its PR references had drifted: entries cited the issue number (filed first, lower) or an earlier stacked-PR number instead of the actual merge-PR number, and six entries had no reference at all. This corrects the section so every entry ends with its real merge PR.

  • 18 references corrected — e.g. #2803#2812, #2781#2802, #2789#2810, #2780#2798/#2799/#2800, #2808#2809, #2791#2794, #2790#2793, #2779#2783, #2773#2776, #2792#2795; and 6 ref-less entries now carry #2807 / #2805 / #2787 / #2786 / #2782 / #2774.
  • 5 missing user-facing entries added#2814 (native apt/yum repos), #2804 (Migration.cfc command alias sweep), #2813 (skip spurious commit after rollback in migrateIndividual), #2777 + #2801 (Linux install-URL & migrator-table doc fixes).
  • Intentionally omitted (no user-facing surface) — #2797 #2772 #2771 #2770 (CI/chore/blog) and #2788 (internal bucket-repo rename, subsumed by #2814).

Verified by grep that nothing belonging to 4.0.2 was misfiled in the # [4.0.1] block, so no entries were moved. Existing entry prose is unchanged except for the trailing reference; the 5 new entries were written from verified commit bodies. Terminal punctuation follows the file's dominant convention — (#NNNN) with no trailing period (167 vs 12 across the file).

Test plan

  • CI commitlint accepts the docs(changelog): header
  • Changelog / sites/guides build unaffected (Markdown-only change)
  • Spot-check that each of the 24 entries resolves to its correct merge PR on GitHub

🤖 Generated with Claude Code

…leased]

Entries in [Unreleased] cited issue numbers (or earlier stacked-PR
numbers) rather than their merge-PR numbers, and six entries carried no
reference at all. Correct 18 references to the actual merge PRs and add
the five missing user-facing entries: #2814 (native apt/yum repos),
#2804 (Migration.cfc command alias sweep), #2813 (skip spurious commit
after rollback in migrateIndividual), and #2777 / #2801 (Linux install
URL and migrator-table doc fixes).

No existing prose changed beyond the trailing reference, and nothing was
misfiled in the 4.0.1 section (verified by grep). Internal-only
CI/chore/blog PRs (#2797, #2772, #2771, #2770) and the user-invisible
bucket rename (#2788) are intentionally omitted.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Peter Amiri <peter@alurium.com>
@github-actions github-actions Bot added the docs label May 27, 2026

@wheels-bot wheels-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wheels Bot — Reviewer A

TL;DR: Pure CHANGELOG.md hygiene -- 18 reference corrections (issue/stacked-PR numbers to actual merge-PR numbers) plus 5 missing user-facing entries for the 4.0.2 changeset. Every reference spot-checked against its merged PR title; all verified. Approving.

Docs

Spot-checked all 23 modified or added references against GitHub:

Entry Old ref New ref Verified title
t.primaryKey() aliases #2803 #2812 feat(migrator): t.primaryKey() columnName/columnNames aliases
t.references() aliases #2781 #2802 feat(migrator): t.references() columnNames alias + underscore flag
Migration.cfc command sweep (none) #2804 feat(migrator): Migration.cfc command consistency sweep
Advisory upgrade-check scaffolding (none) #2805 refactor(cli): add advisory tier to wheels upgrade check scanner
Upgrade-check advisory checks (none) #2807 feat(cli): underscore-references opt-in advisories in wheels upgrade check
wheels_migrator_versions columns #2780 #2800 feat(migrator): enrich wheels_migrator_versions with name + applied_at
model() inside migration tx fix #2789 #2810 fix(model): bypass nested cftransaction when migrator owns the outer
migrateIndividual spurious commit (none) #2813 fix(migrator): skip transaction commit after rollback in migrateIndividual()
Upgrade-check config scan fix #2808 #2809 fix(cli): wheels upgrade underscore-flag pre-check scans all of config/
migrate latest orphan fix #2780 #2798 fix(migrator): handle orphan versions in shared dev databases
Namespace route rejection #2791 #2794 fix(mapper): reject redundant namespace prefix in to= and controller=
WheelsTest auto-bind globals #2790 #2793 fix(test): auto-bind include-injected globals into WheelsTest spec scope
SELECT clause quoting (none) #2787 fix(model): quote column identifiers in SELECT clause builder
packages install alias (none) #2786 fix(cli): make wheels packages install a real alias for add in dispatch
BrowserTest NotWired error (none) #2782 fix(test): BrowserTest reports unwired this.browser with browserDescribe() hint
BrowserTest base URL #2779 #2783 fix(test): resolve BrowserTest base URL through layered lookup at instance time
Linux package nesting #2773 #2776 fix(ci): stop double-nesting framework inside Linux .deb/.rpm packages
onError guard (none) #2774 fix(cli): guard application.wo in onError so init failures don't cascade
?reload=true global change #2792 #2795 fix(events): re-include app/global/*.cfm on bare ?reload=true when files change
Native apt/yum repos (none) #2814 feat(distribution): land Phase 2 native apt/yum repos via Cloudflare R2
Linux install-URL fix (none) #2777 docs(web/guides): correct Linux bleeding-edge install URLs to wheels-be_*
schema_migrations rename (none) #2801 docs: propagate schema_migrations to wheels_migrator_versions rename to sibling v4 docs

The unchanged test-local.sh entry (#2796) and the intentional omissions (#2797, #2772, #2771, #2770 CI/chore/blog; #2788 bucket rename subsumed by #2814) all check out.

Commits

Single commit docs(changelog): correct drifted PR refs and fill 4.0.2 gaps in [Unreleased] -- valid docs type, 77-char subject within the 100-char limit, not ALL-CAPS, body explains the why, DCO Signed-off-by trailer present.

@wheels-bot

wheels-bot Bot commented May 27, 2026

Copy link
Copy Markdown
Contributor

Wheels Bot — Reviewer B (round 1)

A's review is substantive and the approval verdict is correct. One minor incompleteness in the spot-check table, but it does not affect the correctness of the underlying change.

Sycophancy

None detected. A backed the approval with a 22-row reference-verification table and explicit commit-message checks — not a bare "LGTM."

False positives

None detected. Every row I cross-checked against the diff matched: old ref, new ref, and entry prose all align.

Missed issues

One gap: A claims "Spot-checked all 23 modified or added references" but the table contains 22 entries. The missing row is the Three new wheels migrate subcommands entry in the ### Added section, corrected from (#2780) to (#2799) (diff lines 12→18). A verified the other two #2780 corrections — →#2798 (Fixed: migrate latest orphan fix) and →#2800 (Changed: wheels_migrator_versions columns) — but did not list this third one. The correction itself is present and accurate in the diff; this is a completeness gap in A's accounting, not a substantive error.

Verdict alignment

Approve is the right call for a documentation-only PR with no code, migration, or security surface.

Convergence

Aligned. The missing table row is a bookkeeping omission; the actual change is correct and the approval is warranted. Joint recommendation: approve and merge.

@bpamiri

bpamiri commented May 27, 2026

Copy link
Copy Markdown
Collaborator Author

No code change needed — Reviewer B's flagged gap is in Reviewer A's verification table (22 rows vs the 23 claimed), not in the diff. The #2780#2799 correction on the Three new wheels migrate subcommands entry is present and accurate at CHANGELOG.md L30 (ends (#2799), with #2780/#2798 retained as intentional inline cross-references). Both reviewers converged on approve, so there's nothing to push.

@bpamiri bpamiri merged commit 4a6bf7e into develop May 27, 2026
7 checks passed
@bpamiri bpamiri deleted the docs/changelog-unreleased-4-0-2 branch May 27, 2026 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant