-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Stop using detachedConfiguration().extendsFrom in ConditionalDependenciesEnabler #48551
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?
Stop using detachedConfiguration().extendsFrom in ConditionalDependenciesEnabler #48551
Conversation
I did a small smoke test to compare the contents of
for 3 conditional dependencies: Here is the output of code in main:
here is my output
I'm not really sure whether that's worth investigating since such dependencies will be filtered out at |
Thanks @reaver585 I'll have a look into those differences |
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.
This is very cool that you and @Eng-Fouad are looking at this, thanks!
5e157a1
to
1409230
Compare
Status for workflow
|
extendsFrom
deprecation warnings #44660