revert: restore original PythonBot workflow #1826
Conversation
…tability Signed-off-by: Abhijeet Saharan <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet Saharan <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet Saharan <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
|
Hi @exploreriii 🙂 |
WalkthroughDeletes the Node.js workflow notifier and replaces it with inline Bash steps in the GitHub Actions workflow; updates workflow inputs, permissions, concurrency, and PR-resolution/posting logic; adds a CHANGELOG entry noting the revert. Changes
Sequence Diagram(s)sequenceDiagram
participant GHA as GitHub Actions<br/>Workflow
participant GHCLI as GitHub CLI (gh)
participant API as GitHub API
GHA->>GHCLI: Determine PR for HEAD_BRANCH (gh pr list)
GHCLI->>API: List PRs / query PRs
API-->>GHCLI: PR number
GHCLI-->>GHA: PR number
GHA->>GHCLI: List comments on PR (gh pr view / gh api)
GHCLI->>API: Get comments
API-->>GHCLI: Comments list
GHCLI-->>GHA: Comments data
alt Identical comment exists
GHA->>GHA: Skip posting (idempotent)
else
GHA->>GHCLI: Post comment on PR (gh api or gh pr comment)
GHCLI->>API: Create comment
API-->>GHCLI: Comment created
GHCLI-->>GHA: Success
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #1826 +/- ##
=======================================
Coverage 93.29% 93.29%
=======================================
Files 141 141
Lines 9118 9118
=======================================
Hits 8507 8507
Misses 611 611 🚀 New features to boost your workflow:
|
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
|
Thank you! @Abhijeet2409 |
Summary
This PR reverts the PythonBot workflow failure notifier to its original
implementation to restore the previous stable behaviour.
Why
The updated workflow was failing when attempting to post PR comments.
Maintainers requested reverting the workflow to restore stability.
Testing
Fixes #1825