Skip to content

Test companion pub content-size check in the release verifier (#1861 sweep 2) - #1889

Closed
leehack wants to merge 1 commit into
mainfrom
test/1861-sweep-2b
Closed

leehack wants to merge 1 commit into
mainfrom
test/1861-sweep-2b

Conversation

@leehack

@leehack leehack commented Nov 10, 2026

Copy link
Copy Markdown
Owner

Summary

Production-readiness scope

  • User-facing scope: None; tests and error plumbing only.
  • Supported platforms/paths: release tooling
  • Unsupported or intentionally unavailable paths: N/A
  • Out of scope / follow-ups: Refs #1861 (tick after merge); plan #1874.

Test Plan

  • dart format --output=none --set-exit-if-changed and dart analyze on touched dirs
  • TMPDIR=$(mktemp -d) dart test -p vm -j 1 test/unit/tooling/pub_content_limits_test.dart (13 pass)
  • Full CI: stacked or not on main, run via workflow_dispatch of ci.yml (see comment)

Review Notes

  • Mutants: limiting the verifier to the core package fails the three new companion tests

High-risk regression review

  • Classification: high-risk (releaseAutomation)
  • Implementation task: this PR
  • Independent blocking QA task: pending

@leehack

leehack commented Nov 10, 2026

Copy link
Copy Markdown
Owner Author

Landed on main via batch integration PR #1895 (rebase merge, main ba4e0e1b; tree identical to the audited integration head 5ffd1224).

@leehack leehack closed this Nov 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Chat app preview removed for leehack/llamadart-chat-pr-1889.

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