Skip to content

ci(dependabot): drop the ruff group, so ruff updates with the other Python packages - #226

Closed
AlexS-scale wants to merge 3 commits into
mainfrom
AlexS/dependabot-ruff-own-pr
Closed

AlexS-scale wants to merge 3 commits into
mainfrom
AlexS/dependabot-ruff-own-pr

Conversation

@AlexS-scale

@AlexS-scale AlexS-scale commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

What and why

#214 added a ruff Dependabot group, so a ruff release would get a pull request of its own. We'd rather have every Python update in one pull request, so this removes that group and its comment. Ruff goes back into python-minor-and-patch with everything else.

Leaving the group in place isn't harmless. It never took effect: Dependabot refreshed #218 after #214 merged and still bumped ruff there. A full scheduled run could still open a separate ruff pull request on top of that, which would bump ruff in two pull requests at once.

The trade-off: ruff is pinned to an exact version, so a ruff release that reports new findings fails unit in the group pull request, and the lint has to be fixed there before the group can merge. It never breaks main by surprise. Ruff 0.17.0, which #218 bumps to, passes on main today.

How it was tested

The file parses, and the uv entry has one group, python-minor-and-patch. Compared with main, the diff only removes the ruff group and its comment.

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge for the final commit’s stated goal of updating Ruff with the other dependencies.

Summary

Removes the separate ruff group from Dependabot. Ruff minor and patch updates now join python-minor-and-patch.

  • Dependabot no longer has a separate Ruff update group.

Reviews (3) · Last reviewed commit: "ci(dependabot): drop the ruff group inst..." · Reviewed by Greptile

The ruff group alone did not split ruff out: the refresh of the open group pull request
after the change still bumped ruff with everything else.
@AlexS-scale
AlexS-scale requested a review from a team as a code owner October 15, 2026 21:12
@AlexS-scale AlexS-scale changed the title ci(dependabot): keep ruff out of the python-minor-and-patch group ci(dependabot): drop the ruff group, so ruff updates with the other Python packages Oct 15, 2026
@AlexS-scale

Copy link
Copy Markdown
Contributor Author

Closing: we're keeping the current Dependabot config as it is.

@AlexS-scale
AlexS-scale deleted the AlexS/dependabot-ruff-own-pr branch October 15, 2026 21:33
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