Skip to content
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

🚀 Feature: Renovate config use preset config:recommended or config:best-practices #1455

Closed
2 tasks done
rubiesonthesky opened this issue Apr 7, 2024 · 4 comments · Fixed by #1462
Closed
2 tasks done
Labels
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request

Comments

@rubiesonthesky
Copy link
Contributor

Bug Report Checklist

Overview

I propose that renovate config would use present config:recommended or config:best-practices. https://docs.renovatebot.com/upgrade-best-practices/#general-recommendations General recommendations says to use config:best-practices instead of config:recommended, but even config:recommended would be good to have.

This would group monorepos etc together and lessen the spam that renovate easily generates. :)

Additional Info

(I was not sure should this go under feature or tooling, but since this will affect created repositories, I guess this is feature?)

@rubiesonthesky rubiesonthesky added the type: feature New enhancement or request label Apr 7, 2024
@JoshuaKGoldberg
Copy link
Owner

JoshuaKGoldberg commented Apr 8, 2024

Ah, yeah, that would be great. What roughly happened was:

  1. I started looking at Renovate and found the docs to be confusing to read through (Provide a way to see full resolution of presets in the docs? renovatebot/renovate#24448 & links)
  2. Out of "developer efficiency" (read: not wanting to read through all them), I just went with a minimally sized config to start
  3. Later, the kind Renovate folks explained it all during that discussion's linked ESLint work

...so this is kind of a step 4: taking the best practices into this repo. Yes please!

guess this is feature?

Yup! Correct 👍

@JoshuaKGoldberg JoshuaKGoldberg added the status: accepting prs Please, send a pull request to resolve this! label Apr 8, 2024
@JoshuaKGoldberg
Copy link
Owner

@all-contributors please add @rubiesonthesky for ideas.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙

Copy link
Contributor

@JoshuaKGoldberg

I've put up a pull request to add @rubiesonthesky! 🎉

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

JoshuaKGoldberg pushed a commit that referenced this issue Apr 8, 2024
Adds @rubiesonthesky as a contributor for ideas.

This was requested by JoshuaKGoldberg [in this
comment](#1455 (comment))

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
JoshuaKGoldberg added a commit that referenced this issue Apr 26, 2024
## PR Checklist

- [x] Addresses an existing open issue: fixes #1455
- [x] That issue was marked as [`status: accepting
prs`](https://github.com/JoshuaKGoldberg/create-typescript-app/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22)
- [x] Steps in
[CONTRIBUTING.md](https://github.com/JoshuaKGoldberg/create-typescript-app/blob/main/.github/CONTRIBUTING.md)
were taken

## 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 added `maintainLockFilesWeekly` and
`replacements:all`. I'm not sure if `maintainLockFilesWeekly` is needed
when there is pnpm dedupe in every PR already.

- maintainLockFilesWeekly: "Run lock file maintenance (updates) early
Monday mornings."
https://docs.renovatebot.com/presets-default/#maintainlockfilesweekly
- replacementsall: "Apply crowd-sourced package replacement rules."
https://docs.renovatebot.com/presets-replacements/#replacementsall

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/ )

---------

Co-authored-by: Josh Goldberg ✨ <[email protected]>
Copy link

🎉 This is included in version v1.59.0 🎉

The release is available on:

Cheers! 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request
Projects
None yet
2 participants