Skip to content

style: rustfmt files from merges that skipped CI - #4583

Closed
ScriptedAlchemy wants to merge 1 commit into
masterfrom
fix/rustfmt-after-unchecked-merges
Closed

style: rustfmt files from merges that skipped CI#4583
ScriptedAlchemy wants to merge 1 commit into
masterfrom
fix/rustfmt-after-unchecked-merges

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Oct 27, 2026 •
edited by devin-ai-integration Bot
Loading

Copy link
Copy Markdown
Owner

Problem

#4567 and #4573 were admin-merged before CI ran. They put two files on master that fail //:rustfmt_check:

  • crates/tracedecay/src/daemon/hook_v2_replay_consumer.rs
  • crates/tracedecay-sessions/src/runtime/hosts/pi.rs

Because of that, Repository gates fails on every open PR, including ones that don't touch these files (e.g. #4581).

Fix

Ran rustfmt on both files. Formatting only.

Verification

bazel build //:rustfmt_check passes locally.

🤖 Generated with Claude Code

Devin Review

@ScriptedAlchemy @claude
style: rustfmt files from merges that skipped CI
f4af85c 
#4567 and #4573 were admin-merged before CI ran, landing two files that
fail the //:rustfmt_check gate. Every open PR's Repository gates now
fails on code it does not touch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ScriptedAlchemy
ScriptedAlchemy enabled auto-merge (squash) October 27, 2026 07:34
@changeset-bot

changeset-bot Bot commented Oct 27, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f4af85c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

devin-ai-integration[bot]
devin-ai-integration Bot reviewed Oct 27, 2026
View reviewed changes

@devin-ai-integration devin-ai-integration 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@ScriptedAlchemy

ScriptedAlchemy commented Oct 27, 2026

Copy link
Copy Markdown
Owner Author

Superseded: #4578 landed the same rustfmt fix for both files, and master now passes //:rustfmt_check.

auto-merge was automatically disabled October 27, 2026 10:58

Pull request was closed

@ScriptedAlchemy
ScriptedAlchemy deleted the fix/rustfmt-after-unchecked-merges branch October 27, 2026 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ScriptedAlchemy