Skip to content

Drop a ledger reference from the page property values migration comment - #729

Merged
howdu merged 1 commit into
mainfrom
fix/migration-comment-ledger-ref
Oct 2, 2026
Merged

howdu merged 1 commit into
mainfrom
fix/migration-comment-ledger-ref

Conversation

@howdu

@howdu howdu commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

App #1262 removed a ledger reference from the App's published copy of 2026_09_04_000007_create_page_property_values_table. The Core original still carried it, so the two copies differed and PublishedPackageMigrationTest failed in the App backend lane. This is a comment-only change; the schema is unchanged.

Verified: the App's PublishedPackageMigrationTest passes against this branch through the overlay (it failed on main). No other Core migration carries a ledger reference. Hosted CI is billing-blocked; merging on local verification.

The App's published copy of this migration had the reference removed (App #1262), so the copies differed and PublishedPackageMigrationTest failed in the App backend lane. Comment-only; the schema is unchanged.
@howdu
howdu enabled auto-merge (squash) October 2, 2026 22:34
@howdu
howdu merged commit df566e8 into main Oct 2, 2026
31 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