-
-
Notifications
You must be signed in to change notification settings - Fork 2
feat: add renovate #1
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
base: main
Are you sure you want to change the base?
Conversation
Given that there's nothing in the repo yet, maybe we should wait before activating Renovate for this repo? We don't really know what we need to update yet. |
Based on my understanding, renovate should be activated to repositories using this configuration file (e.g. eslint/eslint, which has been enabled), rather than this repository itself. For easier maintenance, we could enable it across the entire eslint org. Alternatively, we might choose to enable it selectively only for repositories that actually require it. |
Yes, this is how renovate is setup for repositories, but as I said, this repository doesn't have any dependencies. It doesn't make sense to set up renovate until we know what we want it to update. There's no way to know that right now. |
It's not clear to me. the file is just mean to be used by other repos, as you can see here: eslint/eslint#19907 |
Oh I see, you're intending to share this configuration across repos. I think we should hold off on this for now because I'm not sure what we'd consider common for all the repos. |
I think starting with common GitHub Actions workflows or Renovate config setups would be a good approach for this repository. When reviewing some of the issues I suggested earlier (such as the ones below), these workflows and logics seem to be commonly used across ESLint repositories:
On a related note, there are currently three repositories that serve to maintain centralized workflows or templates:
Clarifying the scope of these repositories and consolidating common logic into these three centralized repositories could help make maintaining the ESLint project easier. (Just as a personal opinion, I think clarifying the scope is important. For example, Also, I think enabling Renovate or Dependabot across repositories would be helpful, since it would automate updates and allow us to quickly catch any potential incompatibilities with new versions. |
I agree. What I'm saying is that we haven't decided on what common Renovate settings we want across repos, so I think it would be better to open an issue to discuss that before making a pull request. Similarly, I think we should look across repos to see what are the most common GitHub action workflows we have and start by adding those to this repo before we look at creating things that don't necessarily exist already. |
Ah, I got it! 😄 I've been interested in this topic for a while, so I'll go ahead and open an issue about it first. |
I've finished writing an issue here: #4 |
Hi, @aladdin-add 😄 There has been some progress on this issue, along with discussions about how the team would like to move forward: I think we're ready to proceed with this PR. If you have a chance, it would be great if you could take a look. |
I'd like to see a concrete proposal for what settings make sense for all repos. |
If @aladdin-add doesn’t have time, and if he’s okay with it, I’d be happy to take a look after the |
I'm sorry, I've been recovering from an illness recently and have very limited time to devote to open-source projects. But I'll try to get this done within 1-2 weeks. If you have time to help, I'd be happy to have your assistance. |
e295914
to
08b61e9
Compare
1. base.json5 is applicable to all projects 2. eslint-base.json5 extends base.json5 and adds ESLint related configurations
08b61e9
to
8f9e81a
Compare
description: "Update ESLint packages together.", | ||
groupName: "eslint", | ||
matchPackagePrefixes: ["@eslint/"], | ||
matchPackageNames: ["espree", "eslint-scope", "eslint-visitor-keys"], |
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.
not adding eslint
, eslint-config-eslint
- they usually belong to dev-dependencies.
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.
pushed a new commit to group eslint
& eslint-config-eslint
: 360397d
@@ -0,0 +1,19 @@ | |||
{ | |||
$schema: "https://docs.renovatebot.com/renovate-schema.json", | |||
extends: ["github>eslint/workflows//.github/renovate/base.json5"], |
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.
Can we just use a local file path instead?
extends: ["github>eslint/workflows//.github/renovate/base.json5"], | |
extends: ["base.json5"], |
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.
not sure if it can work as expected - it may be seen as built-in preset. I will change it to ./base.json5
to see if it's working.
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.
FYI, I've also looked at the official documentation, but it seems there's no way to use a local path directly. 🤔
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.
@@ -0,0 +1,19 @@ | |||
{ | |||
$schema: "https://docs.renovatebot.com/renovate-schema.json", | |||
extends: ["github>eslint/workflows//.github/renovate/base.json5"], |
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.
FYI, I've also looked at the official documentation, but it seems there's no way to use a local path directly. 🤔
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.
Re-approving.
base.json5
is applicable to all projects.eslint-base.json5
extends base.json5 and adds ESLint related configurations.fixes: #13