-
Notifications
You must be signed in to change notification settings - Fork 10.3k
chore: simplify Renovate setup #39339
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
chore: simplify Renovate setup #39339
Conversation
const packageRules = new Map() | ||
|
||
const globalPackageRules = [ | ||
// bundle well known monorepos |
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.
renovate does all this out of the box now... I assume this predates that
// there is no excludeMatchSourceUrlPrefixes option so we force babel to be disabled | ||
const globalExcludePackages = [] | ||
const globalExcludePackagePatterns = [`^@babel`] |
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.
🤷🏼 it doesn't say why. yolo.
`:separateMajorReleases`, | ||
`:combinePatchMinorReleases`, | ||
`:ignoreUnstable`, |
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.
these are in the recommended config
dependencyDashboardApproval: true, | ||
}, | ||
dependencyDashboard: true, | ||
ignoreDeps: [`react`, `react-dom`, `uuid`, `gatsby-interface`], |
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.
? unclear why. dropping.
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.
I think we just need to try it and see how it behaves. If something is very wrong we can revert (or adjust) and at least learn "why" this setup is/was so convoluted. AFAIK there is no auto-merge for renovate PRs, so things still need to be checked by someone at least for now before being merged which gives a chance to verify things are ok
Replace massive, complicated generated Renovate config and script that generates it with a simple config that:
Hopefully this will make Renovate PRs less noisy and consistently actionable.
✅
npx --package renovate renovate-config-validator