Add --doc, --sources, and --transitive options to publishLocal command #3512
Conversation
| localIvyRepo: String = null, | ||
| sources: Boolean = true, | ||
| doc: Boolean = true, | ||
| transitive: Boolean = true |
There was a problem hiding this comment.
I think the default for transitive should be false. Also, it should include compileModuleDeps as well, when true.
There was a problem hiding this comment.
I agree that transitive should default to false. I ran into issues with it and some publishLocal tasks aggregation via T.traverse done elsewhere (because of duplicated artifacts it seems). Although starting from a blank slate, a default to true would have been nicer IMO.
There was a problem hiding this comment.
About compileModuleDeps, I'm not sure. My aim with --transitive is to allow users to use something they published straightaway. So the standard dependencies need to be published too, but the compile-only ones aren't needed. Users can resolve the published module fine without publishing the compile-only dependencies.
Fixes #2487
I'd like to use that locally, but keeping this PR as a draft until tests can be added, later on. (Bin compat might need to be taken into account too.)