Skip to content

MNT: Add Renovate config to master so it actually takes effect#1039

Merged
Gui-FernandesBR merged 1 commit into
masterfrom
worktree-renovate-config-master-fix
Jun 27, 2026
Merged

MNT: Add Renovate config to master so it actually takes effect#1039
Gui-FernandesBR merged 1 commit into
masterfrom
worktree-renovate-config-master-fix

Conversation

@Gui-FernandesBR

@Gui-FernandesBR Gui-FernandesBR commented Jun 27, 2026

Copy link
Copy Markdown
Member

Why

Renovate (Mend hosted app) reads its repository configuration only from the default branch, which for this repo is master. The config added in #972 was merged into develop, so it never took effect.

Running with bare defaults is why Renovate has been misbehaving:

  • raising PRs against master instead of develop,
  • not grouping non-major updates,
  • and naively bumping every pinned line in docs/requirements.txt — which is autogenerated by pip-compile and carries a "DO NOT EDIT" header. This produced ~50+ noisy PRs.

What this does

Adds .github/renovate.json on master with the intended settings, plus the docs fix:

  • baseBranches: ["develop"] — future dependency PRs target develop (the original intent).
  • Grouping of non-major updates, dependencies label, dependency dashboard (unchanged from MNT: Configure Renovate Bot targeting develop branch #972).
  • Enables the pip-compile manager scoped to docs/requirements.txt. Its header records pip-compile docs/requirements.in, so Renovate regenerates the lock file properly instead of patching pinned lines.
  • Disables pip_requirements for docs/requirements.txt only, to avoid duplicate PRs. The hand-written requirements.txt, requirements-optional.txt, and requirements-tests.txt are still managed normally.

Follow-up

The ~54 open Renovate PRs that only edit docs/requirements.txt are being closed separately; once this merges they won't be recreated. The develop copy of renovate.json should be synced to match (or will reconcile on the next release merge).

Renovate (Mend hosted app) reads repository configuration only from the
default branch, which is `master`. The config introduced in #972 was merged
into `develop`, so it never took effect: Renovate ran with bare defaults,
raised PRs against `master` instead of `develop`, did not group updates, and
naively bumped every pinned line in the autogenerated `docs/requirements.txt`.

This adds the configuration on `master` and:
- keeps the intended settings (baseBranches: develop, grouping, labels,
  dependency dashboard);
- enables the `pip-compile` manager scoped to `docs/requirements.txt`, which is
  autogenerated from `docs/requirements.in`, so the lock file is regenerated
  properly instead of patched line by line;
- disables the `pip_requirements` manager for that one file to avoid duplicate
  PRs (it still manages the hand-written requirements files).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Gui-FernandesBR Gui-FernandesBR requested a review from a team as a code owner June 27, 2026 02:49
This was referenced Jun 27, 2026
This was referenced Jun 27, 2026
@codecov

codecov Bot commented Jun 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.42%. Comparing base (1e748d8) to head (a74c3aa).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1039      +/-   ##
==========================================
+ Coverage   81.12%   81.42%   +0.30%     
==========================================
  Files         107      107              
  Lines       13881    14026     +145     
==========================================
+ Hits        11261    11421     +160     
+ Misses       2620     2605      -15     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Gui-FernandesBR Gui-FernandesBR merged commit d41dd11 into master Jun 27, 2026
11 checks passed
@Gui-FernandesBR Gui-FernandesBR deleted the worktree-renovate-config-master-fix branch June 27, 2026 03:07
Gui-FernandesBR added a commit that referenced this pull request Jun 27, 2026
Renovate reads its config only from the default branch (master), where the
corrected config is added in #1039. This updates develop's renovate.json to be
byte-identical to that one (pip-compile manager for the autogenerated
docs/requirements.txt; pip_requirements disabled for that file) so master and
develop don't diverge and the next master<->develop merge won't hit an add/add
conflict on .github/renovate.json.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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