-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
feat(tools): use config:recommended for renovate #1462
feat(tools): use config:recommended for renovate #1462
Conversation
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.
Cool, thanks for getting this started! A couple of questions/changes I'm not super confident on. 🧹
Huh. I don't get that locally. If you can reproduce that might be a separate issue? |
Co-authored-by: Josh Goldberg ✨ <[email protected]>
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.
💯 thanks, this is great!
I'm particularly excited to have the recommended replacements automated for the repos. 😄
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.
Almost!
Co-authored-by: Josh Goldberg ✨ <[email protected]>
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.
💯 thanks!
@all-contributors please add @rubiesonthesky for code.
|
I've put up a pull request to add @rubiesonthesky! 🎉 I couldn't determine any contributions to add, did you specify any contributions? |
Adds @rubiesonthesky as a contributor for code. This was requested by JoshuaKGoldberg [in this comment](#1462 (comment)) --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
🎉 This is included in version v1.59.0 🎉 The release is available on: Cheers! 📦🚀 |
## PR Checklist - [ ] Addresses an existing open issue: fixes #000 - [ ] That issue was marked as [`status: accepting prs`](https://github.com/JoshuaKGoldberg/TypeStat/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22) - [x] Steps in [CONTRIBUTING.md](https://github.com/JoshuaKGoldberg/TypeStat/blob/main/.github/CONTRIBUTING.md) were taken ## Overview Same as here: JoshuaKGoldberg/create-typescript-app#1462
PR Checklist
status: accepting prs
Overview
Using
config:recommended
for renovate config.config:best-practices
looked too opinionated for common user, so I think this recommended is better fit. I also addedmaintainLockFilesWeekly
andreplacements:all
. I'm not sure ifmaintainLockFilesWeekly
is needed when there is pnpm dedupe in every PR already.BTW, some tools seems to be fighting over formatting in this repo. When I save this file in VS code it's formatted differently than what is seen in this PR. So when you stage / commit the file, it gets different formatting.
(I re-created this PR because I just learned that using this branch name, you will get renovate validation for free https://docs.renovatebot.com/config-validation/ )