New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add catalog to dependencies to update by Renovate #3465
Conversation
## Description I noticed the Catalog was missing from these updates, but why is it limited to development dependencies?
37714ae
to
7d06b7b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems okay!
I noticed the Catalog was missing from these updates, but also why is it limited to development dependencies?
When you say "it", what are you referring to? Many of the Airflow dependencies are limited by the constraints Airflow provides, so we try not to pin those dependencies directly.
@AetherUnbound I'm refering to that group of dependencies, it includes all the python projects, not only the catalog so it looked strange to me. openverse/.github/renovate.json Lines 67 to 70 in 9b44496
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't the catalog covered by lines 58-61?
@dhruvkb I believe that is only for putting the label on the files that are under the catalog folder 🤔 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
IIRC I excluded the catalog dependencies from that list because I wasn't sure if its development dependencies could get upgraded in a grouped upgrade due to Airflow's constraints on dependency management. I can't think of what the obstacle I imagined to exist was, but it should be fine.
The group modified in this PR is just to group development dependencies, though. Renovate will automatically include the catalogue's requirements.txt files when considering upgrades. It differs from dependabot in this way, you don't need to explicitly include each thing you want renovate to manage. Instead it defaults to managing updates for everything, and you only have to configure excludes, labeling rules, and dependency groups if you want grouped update PRs (instead of an individual PR per dependency).
In other words, it already handles the catalogue's dependencies, it just opens a PR for each one. This will make it so that its development dependencies join the others and get upgraded together in a single PR, but the non-development dependencies will still continue to get the one-PR-per-upgrade treatment that renovate should be giving them already.
Description
I noticed the Catalog was missing from these updates, but also why is it limited to development dependencies?
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin