Skip to content

PR Formatter: Skip PR formatter for depandabot opened PRs#1067

Merged
Prajna1999 merged 2 commits into
mainfrom
chore/fix-pr-formatter-bypass-depandatbot
Jul 23, 2026
Merged

PR Formatter: Skip PR formatter for depandabot opened PRs#1067
Prajna1999 merged 2 commits into
mainfrom
chore/fix-pr-formatter-bypass-depandatbot

Conversation

@Prajna1999

@Prajna1999 Prajna1999 commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Issue

Closes #1068

Summary

  • Before: The PR formatter processed all pull requests, including those opened by Dependabot.
  • Now: The PR formatter skips pull requests opened by Dependabot.
  • Required changes to the PR formatter logic to include checks for the author of the pull request.

Checklist

Before submitting a pull request, please ensure that you mark these task.

  • Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
  • If you've fixed a bug or added code that is tested and has test cases.

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The PR-formatting workflow now skips the format-pr job when the pull request actor is dependabot[bot].

Changes

PR formatter execution

Layer / File(s) Summary
Conditional formatter execution
.github/workflows/pr-formatter.yml
Adds a job-level condition excluding dependabot[bot] from the format-pr job.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: vprashrex, ayush8923

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The workflow now conditionally skips the formatter for dependabot[bot], which satisfies the issue's requirement to run only for human-created PRs.
Out of Scope Changes check ✅ Passed The change is narrowly scoped to the Dependabot guard in the PR formatter workflow and introduces no unrelated behavior.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: skipping the PR formatter for Dependabot-opened pull requests.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/fix-pr-formatter-bypass-depandatbot

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot changed the title PR Formatter: Skip PR formatter is skipped for depandabot opened PRs chore(ci): Skip Dependabot PR formatting Jul 23, 2026
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

OpenAPI changes   ⚪ No API surface changes

Note

This PR does not modify the API contract.

main49fd4ad5 · generated by oasdiff

@Prajna1999 Prajna1999 linked an issue Jul 23, 2026 that may be closed by this pull request
@Prajna1999 Prajna1999 changed the title chore(ci): Skip Dependabot PR formatting PR Formatter: Skip PR formatter for depandabot opened PRs Jul 23, 2026
@Prajna1999 Prajna1999 self-assigned this Jul 23, 2026
@Prajna1999 Prajna1999 added bug Something isn't working ready-for-review labels Jul 23, 2026
Comment thread .github/workflows/pr-formatter.yml Outdated
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Ayush8923 Ayush8923 removed bug Something isn't working ready-for-review labels Jul 23, 2026
@Ayush8923

Copy link
Copy Markdown
Collaborator

I removed all the labels after approving the PR. The semantic release workflow automatically adds the release related label when it runs, and the other labels are no longer needed after the PR is merged. In particular, the ready for review label doesn’t make sense once the PR has been approved and merged.

@Prajna1999
Prajna1999 merged commit 49fd4ad into main Jul 23, 2026
5 of 6 checks passed
@Prajna1999
Prajna1999 deleted the chore/fix-pr-formatter-bypass-depandatbot branch July 23, 2026 07:49
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.4.0-main.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.4.0-main.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR Formatter: Conditional running for Dependabot

3 participants