Skip to content

emit: cross-module use emission (E0425 imports) - #5398

Closed
gunbai-bot[bot] wants to merge 3 commits into
mainfrom
session/cool-carp-66
Closed

gunbai-bot[bot] wants to merge 3 commits into
mainfrom
session/cool-carp-66

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Auto-opened by session-dashboard for session cool-carp-66.
Pushing to session/cool-carp-66 advances this PR.

Worker attestation

Before flipping this PR to ready for review, confirm each item:

  • Title describes the change (not the session id or branch).
  • PR body summarises what and why (replace the TODO below).
  • Tests run: name the command (e.g. npm test, cargo test) and the result.
  • If this closes a work item, the body contains a Closes #N directive.
  • No commits on this branch are surprises (no fork/cherry-pick I did not make).
  • No secrets / credentials / large binaries staged.

Summary

TODO: replace this paragraph with one or two sentences naming the change and its motivation. Reviewers read this first.

Test plan

  • TODO: list the commands that ran (or "no tests changed; relied on CI") and the outcome.

@gunbai-bot

gunbai-bot Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor Author

Closing: this PR commits the full regenerated seed (70 files, incl. unrelated v2/std/reducible.dag deletions) with no .dag emitter change, so it cannot land under the lockstep guardrails (committed seed must stay main's until the green co-land in #5325). Your E0425 diagnosis (SemVerConstraint / List container types leaking into the use-set) is sound and is being authored as an emitter .dag fix in the integration branch. Thank you for the root-cause.

@gunbai-bot gunbai-bot Bot closed this Jun 20, 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