paparazzi icon indicating copy to clipboard operation
paparazzi copied to clipboard

Allow custom directories for the snapshot outputs

Open JavierSegoviaCordoba opened this issue 2 years ago • 2 comments

Instead of only allowing src/main/, other projects, including official JetBrains products, are using a more flattened project structure, like src (without main).

A paparazzi Gradle extension which can have a sourcesDir property or whatever name should be great.

JavierSegoviaCordoba avatar Oct 12 '22 23:10 JavierSegoviaCordoba

How would this work with source sets? Snapshot outputs wouldn't go in src/main along with source files, but rather src/test.

Also, I'd refrain from a Gradle extension which is more for per-project/module configuration (that would mean either using subprojects {} in a legacy structure or a convention plugin which sets this property on every project using Paparazzi) and instead use a global system property.

jrodbx avatar Oct 12 '22 23:10 jrodbx

Official JetBrains libraries are using test directly instead of src/test.

I wouldn't use allprojects/subprojects as they break project isolation and are a bad practice in Gradle.

I would suggest using convention plugins, but that is not related to paparazzi itself, which only needs to support a custom directory output, how the user decides to share the configuration across all modules is up to them, as they are already doing when they set the sources directories with KGP.

JavierSegoviaCordoba avatar Oct 12 '22 23:10 JavierSegoviaCordoba