Possibly conflicting information regarding gradle subprojects

[mgroth0]

Possibly conflicting information regarding gradle subprojects

[jeremylong]

The plugin will scan only the projects it is applied to. If you apply it to the root it will scan everything - but each project will get its own report unless you use the aggregate task

[jeremylong]

There is no contradiction. You can apply the plugin to an entire multi-project build and only scan a specific project by specifying scanProjects

[jeremylong]

Or you can only apply the plugin to a specific build. Multiple ways to achieve the same goal.

[guai]

this line in the docs is totally misleading "scanProjects - A list of projects that will be scanned, all other projects are skipped. The list or projects to skip must include a preceding colon: scanProjects = [':app']. This is mutually exclusive with the skipProjects property."
and in reality this is a filter, what projects to include from the list of subprojects
killed half-a-day on it, then looked into sources

[mgroth0]

Thank you for clarifying, @jeremylong

The plugin will scan only the projects it is applied to. If you apply it to the root it will scan everything

I think this breaks idiomatic Gradle norms and is not future-proof. Gradle plugins applied to the root do not typically have a direct effect on subprojects. Also, the newset gradle versions are moving towards project isolation, which means that no projects (including the root project) should be configuring other projects like this. Sharing build logic between projects is meant to be done through plugins.

So I think there are two issues

1. The documentation can be more clear. I can offer a documentation PR addressing where I was confused if that is helpful. I would just add a couple of sentances providing the answers you did here.
2. Remove the ability for the root project to configure sub projects; this is needed for compatibility with project isolation in future gradle versions. Shall I create a new issue for this?

[jeremylong]

Sharing build logic between projects is meant to be done through plugins.

Do you mean conventions?

Also - if you don't mind I would love a PR to update the documentation.