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

Prepare discovery avoidance work #635

Open
wants to merge 44 commits into
base: master
Choose a base branch
from

Conversation

lacasseio
Copy link
Member

No description provided.

Signed-off-by: Daniel Lacasse <[email protected]>
The created state does not have any meaning anymore.

Signed-off-by: Daniel Lacasse <[email protected]>
Signed-off-by: Daniel Lacasse <[email protected]>
Signed-off-by: Daniel Lacasse <[email protected]>
If onwers has baseName, use their value, else use elementName.

Signed-off-by: Daniel Lacasse <[email protected]>
Signed-off-by: Daniel Lacasse <[email protected]>
This is not the exact solution we want but in term of full discovery
it's ok. We will want to move toward an incremental discovery when we
support `named` API.

Signed-off-by: Daniel Lacasse <[email protected]>
Instead of computing the lookup result eagerly, we will allow processing
additional entities as they become available.

Signed-off-by: Daniel Lacasse <[email protected]>
The Java plugin eagerly creates the compile tasks which circumvent the
task discovery by fully qualified name.

Signed-off-by: Daniel Lacasse <[email protected]>
After project evaluation, Gradle compute a metadata list of all outgoing
artifacts. We need to ensure all outgoing Configuration are present.

Signed-off-by: Daniel Lacasse <[email protected]>
Signed-off-by: Daniel Lacasse <[email protected]>
Signed-off-by: Daniel Lacasse <[email protected]>
@lacasseio lacasseio force-pushed the lacasseio/prepare-discovery-avoidance branch from 39c0ea9 to 8d490ee Compare June 22, 2022 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant