Skip to content

ci(deps): bump lower bound on pip dependabot PRs#40308

Merged
hainenber merged 1 commit into
masterfrom
ci/dependabot-pip-versioning-strategy
May 21, 2026
Merged

ci(deps): bump lower bound on pip dependabot PRs#40308
hainenber merged 1 commit into
masterfrom
ci/dependabot-pip-versioning-strategy

Conversation

@rusackas
Copy link
Copy Markdown
Member

SUMMARY

Follow-up to @villebro's review on #40186 (sqlglot 28→30 bump). Dependabot's default behavior for the pip ecosystem is to only widen the upper bound of constraints in `pyproject.toml`, leaving the lower bound at the old version forever. Over time the lower bound becomes meaningless because no installed environment ever resolves that low.

Setting `versioning-strategy: increase` makes dependabot bump both sides of the range to the new version, e.g. `sqlglot>=28.10.0, <29` → `sqlglot>=30.8.0, <31` in one PR.

BEFORE/AFTER

Take a bump from sqlglot 28.10.0 → 30.8.0:

Before After
Upper bound <29<31 <29<31
Lower bound >=28.10.0 (unchanged) >=30.8.0

The lower bound drift is what @villebro flagged on #40186 — over many bumps it becomes a fiction.

What this PR doesn't do

Lockfile regeneration (`./scripts/uv-pip-compile.sh` against `requirements/base.txt` / `development.txt`) still needs a human or a separate hook. Dependabot for pip doesn't auto-run `uv` and won't update compiled pin files. That's a separate problem to solve later — this PR just fixes the `pyproject.toml`-side drift.

TESTING INSTRUCTIONS

Config-only change. To validate, dependabot's next pip-ecosystem PR will set both bounds.

ADDITIONAL INFORMATION

  • Has associated issue
  • Required feature flags
  • Changes UI
  • Includes DB Migration
  • Introduces new feature or API
  • Removes existing feature or API

🤖 Generated with Claude Code

…: increase)

Follow-up to @villebro's review on #40186. Dependabot's default
behavior for the pip ecosystem is to only *widen the upper bound* of
constraints in `pyproject.toml`, leaving the lower bound at the
old version forever. Over time the lower bound becomes meaningless
because no installed environment ever resolves that low.

Setting `versioning-strategy: increase` makes dependabot bump both
sides of the range to the new version, e.g.
`sqlglot>=28.10.0, <29` → `sqlglot>=30.8.0, <31` in one PR.

Lockfile regeneration (`./scripts/uv-pip-compile.sh`) still needs a
human or a separate hook — dependabot for pip doesn't auto-run uv —
but the pyproject.toml side is now consistent with what's intended.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bito-code-review
Copy link
Copy Markdown
Contributor

bito-code-review Bot commented May 20, 2026

Bito Automatic Review Skipped - Files Excluded

Bito didn't auto-review this change because all changed files are in the exclusion list for automatic reviews. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the excluded files settings here, or contact your Bito workspace admin at evan@preset.io.

@hainenber hainenber merged commit 2cd5efa into master May 21, 2026
63 of 65 checks passed
@hainenber hainenber deleted the ci/dependabot-pip-versioning-strategy branch May 21, 2026 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants