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

Migrate deprecated internal presets #7572

Closed
Tracked by #16359
HonkingGoose opened this issue Oct 27, 2020 · 6 comments
Closed
Tracked by #16359

Migrate deprecated internal presets #7572

HonkingGoose opened this issue Oct 27, 2020 · 6 comments
Labels
core:config Related to config capabilities and presets priority-4-low Low priority, unlikely to be done unless it becomes important to more people type:refactor Refactoring or improving of existing code

Comments

@HonkingGoose
Copy link
Collaborator

HonkingGoose commented Oct 27, 2020

What Renovate type, platform and version are you using?

GitHub hosted app.

Describe the bug

I'm confused about what the correct base configuration is...

The docs for config:base-js say this is deprecated:

{
  "extends": [
    "config:base"
  ]
}

But that's what the Renovate bot gives me with its on-boarding pull requests.

I noticed that there are separate configurations for web-apps and libraries:

Shouldn't the bot try to use either config:js-app or config:js-lib in the on-boarding pull request?

Relevant debug logs

To Reproduce

Take a look at this on-boarding pull request I got for a Docusaurus v2 web app: HonkingGoose/git-gosling@0cff589.
The bot used the deprecated config:base-js template instead of the config:js-app template.
EDIT: Whoops! I totally misread the docs on this part, ignore the strike-through text.

Additional context

I think the Renovate bot already tries to determine if you have a web-app or library, to determine if it should pin your dependencies or not? So maybe this logic can be plugged into the onboarding pull request code?

@JamieMagee
Copy link
Contributor

The PR you shared only shows config:base in renovate.json. This looks correct to me?

I don't think we need to explicity specify config:js-app or config:js-lib, as config:base already has :autodetectPinVersions, which deals with app/lib detections.

@rarkins
Copy link
Collaborator

rarkins commented Oct 27, 2020

config:js-app is deprecated. config:base is not. But actually now presets are internalized we could remove them completely and instead migrate them.

@HonkingGoose
Copy link
Collaborator Author

The PR you shared only shows config:base in renovate.json. This looks correct to me?

The bot uses the right config:base item in renovate.json, so that part was me not reading the docs properly, whoops! 😄


But actually now presets are internalized we could remove them completely and instead migrate them.

I don't really understand what this means, but I'll let you deal with this report how you see fit. 😄 😉

@rarkins rarkins changed the title Docs say config:base-js is deprecated, but Renovate onboarding PR uses deprecated config. Migrate deprecated internal presets Oct 27, 2020
@rarkins
Copy link
Collaborator

rarkins commented Oct 27, 2020

We don't have too many presets mentioned as deprecated:

image

For each of these we should remove them and instead add a migration rule in https://github.com/renovatebot/renovate/blob/master/lib/config/migration.ts

@rarkins rarkins added type:refactor Refactoring or improving of existing code priority-4-low Low priority, unlikely to be done unless it becomes important to more people labels Oct 27, 2020
@HonkingGoose
Copy link
Collaborator Author

We don't have too many presets mentioned as deprecated:

image

For each of these we should remove them and instead add a migration rule in https://github.com/renovatebot/renovate/blob/master/lib/config/migration.ts

@rarkins maybe this issue can be moved from "needs requirements" to "ready" on the Renovate project board? Seeing as you already gave the "recipe" of the changes?

@rarkins
Copy link
Collaborator

rarkins commented Jul 10, 2023

We now don't have any presets with deprecated in their description

@rarkins rarkins closed this as completed Jul 10, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
core:config Related to config capabilities and presets priority-4-low Low priority, unlikely to be done unless it becomes important to more people type:refactor Refactoring or improving of existing code
Projects
None yet
Development

No branches or pull requests

4 participants