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

config: enable renovatebot #59

Merged
merged 1 commit into from
Aug 5, 2024
Merged

config: enable renovatebot #59

merged 1 commit into from
Aug 5, 2024

Conversation

onebytegone
Copy link
Contributor

No description provided.

@onebytegone onebytegone requested a review from crgwbr July 29, 2024 12:29
@crgwbr
Copy link
Contributor

crgwbr commented Aug 4, 2024

@onebytegone Any particular reason why we'd put these settings directly in this repo, vs. making a new (reusable) preset and using that here? The config itself LGTM.

@onebytegone
Copy link
Contributor Author

@crgwbr By preset, do you mean adding something to @silvermine/standardization that all the repos can then use? Eventually, that's the goal. However, since we're not completely sure this is the right config, I wanted to try it out first before putting this in standardization.

@crgwbr
Copy link
Contributor

crgwbr commented Aug 5, 2024

@onebytegone

By preset, do you mean adding something to @silvermine/standardization that all the repos can then use?

Yeah, though the docs recommend putting it in its own repo. Here's an example:

See also, docs: https://docs.renovatebot.com/config-presets/

@onebytegone
Copy link
Contributor Author

@crgwbr Ahhh, makes sense. silvermine/renovate-config has been created and this PR updated.

@crgwbr
Copy link
Contributor

crgwbr commented Aug 5, 2024

Awesome! LGTM

@onebytegone onebytegone merged commit 8fb2e0b into master Aug 5, 2024
18 checks passed
@onebytegone onebytegone deleted the renovatebot branch August 5, 2024 13:29
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.

2 participants