remove old auto-formte implimentaion from ci.yaml#6250
remove old auto-formte implimentaion from ci.yaml#6250YashSuthar983 wants to merge 5 commits intoRustPython:mainfrom
Conversation
Signed-off-by: Yash Suthar <[email protected]>
Co-authored-by: fanninpm <[email protected]>
Signed-off-by: Yash Suthar <[email protected]>
Signed-off-by: Yash Suthar <[email protected]>
Signed-off-by: Yash Suthar <[email protected]>
|
Caution Review failedThe pull request is closed. WalkthroughThe pull request removes the Changes
Sequence DiagramsequenceDiagram
actor GitHub as GitHub<br/>(PR Event)
participant Workflow as PR Auto-format<br/>Workflow
participant Rust as Rust/cargo fmt
participant Git as Git<br/>(Push)
participant API as GitHub API<br/>(PR Comment)
GitHub->>Workflow: Trigger on pull_request_target
Workflow->>Workflow: Check skip conditions<br/>(label, commit message)
alt Skip Detected
Workflow->>Workflow: Exit early
else Proceed
Workflow->>Workflow: Checkout PR branch
Workflow->>Workflow: Wait for status checks
Workflow->>Rust: Setup rustfmt toolchain
Workflow->>Rust: Run cargo fmt --all
Rust-->>Workflow: Format changes detected?
alt Changes Found
Workflow->>Git: Commit and push<br/>formatted code [skip ci]
Git-->>Workflow: Success
Workflow->>API: Post PR comment<br/>(formatting applied)
API-->>Workflow: Comment posted
else No Changes
Workflow->>Workflow: Log: already formatted
end
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes
Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary by CodeRabbit