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(gradle): merge gradle-lite into gradle #10132
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.
I've fixed some type imports 🙃
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
}; | ||
return config.deepExtract | ||
? deep.extractAllPackageFiles(config, packageFiles) | ||
: shallow.extractAllPackageFiles(config, packageFiles); |
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.
That's good it's only can be either deep or shallow, not both. I doubt the reasonable mixed solution is possible.
I'm not sure if this is |
name: 'deepExtract', | ||
description: `Enable extraction of dependencies using package managers.`, | ||
type: 'boolean', | ||
default: false, |
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.
If you're not ready to merge a breaking change yet, I'd recommend setting this to true
, merging this PR as a non-breaking change, and then open another one which is marked as breaking where you change this to false
.
Lowers the risk of getting merge conflicts if you want to wait and collect many breaking changes into a single release.
…le-lite # Conflicts: # lib/manager/gradle/deep/index.spec.ts # lib/manager/gradle/deep/utils.ts
# Conflicts: # lib/manager/gradle-lite/readme.md
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Removes standalone gradle-lite manager and merges it into the gradle manager as the default behavior, while adding deepExtract as an option to re-enable previous behavior. Closes #10091 BREAKING CHANGE: Gradle extraction now defaults to JS-based parsing (previously "gradle-lite")
Changes:
Removes standalone
gradle-lite
manager and merges it into thegradle
manager as the default behavior, while addingdeepExtract
as an option to re-enable previous behavior.Context:
Closes #10091
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: