Skip to content

resolve conflicts: #24644 feat!: forbid external note validation checks - #24876

Closed
PhilWindle wants to merge 1 commit into
fwd-port-24644from
resolve-24644
Closed

resolve conflicts: #24644 feat!: forbid external note validation checks#24876
PhilWindle wants to merge 1 commit into
fwd-port-24644from
resolve-24644

Conversation

@PhilWindle

Copy link
Copy Markdown
Contributor

Automated conflict-resolution attempt for #24863 (fwd-port-24644). Merges next + v5-next PR #24644.

Resolution summary

Two files had committed conflict markers; both hunks were trivial (one side empty), so the resolution is a clean union.

  • docs/docs-developers/docs/resources/migration_notes.md: kept next's ## 5.0.1 heading (incoming side was empty). The renamed-history-helpers note from feat!: forbid external note validation checks #24644 was already present below the conflict and is untouched. Union preserved.
  • yarn-project/txe/esbuild/plugins/size_guard.mjs: kept the incoming bump-log line // - 2026-07-13: bumped total to 14.5 MiB. (next side was empty).

Flag for human review

In size_guard.mjs, the added bump-log comment says the total was bumped to 14.5 MiB, but totalLimitMiB remains at next's value of 14 (that line was outside the conflict and unchanged on both sides). This inconsistency is inherited from the forward-port, not introduced here. Confirm whether totalLimitMiB should be raised to match the log entry, or the log line adjusted.

No generated/pinned artifacts were involved, so no regeneration is required.

Confidence: high on the marker resolution; please verify the size-guard limit intent noted above.

@PhilWindle

Copy link
Copy Markdown
Contributor Author

Superseded by #24918 — the v5-next→next backlog is now one time-ordered forward-port PR with a single resolution PR on top, instead of fragmented per-PR pairs (the fragmentation broke commit dependencies). Closing.

@PhilWindle PhilWindle closed this Jul 22, 2026
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