Skip to content

Extension: Add allowDependency with ExternalModuleDependency overload - #432

Open
hfhbd wants to merge 2 commits into
cashapp:trunkfrom
hfhbd:external
Open

Extension: Add allowDependency with ExternalModuleDependency overload#432
hfhbd wants to merge 2 commits into
cashapp:trunkfrom
hfhbd:external

Conversation

@hfhbd

@hfhbd hfhbd commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

Fixes #428

ExternalModuleDependency is the return type of the new DependencyFactory.create method.

@JakeWharton

Copy link
Copy Markdown
Collaborator

So the reason we haven't done this is that it creates the opportunity for license changes to go entirely unnoticed. The Maven coordinate allowance was meant to be more difficult to use because it was a last resort.

This has come up before… somewhere.

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.

allowDependency: Add ExternalModuleDependency overload

2 participants