dweiss commented on PR #14764: URL: https://github.com/apache/lucene/pull/14764#issuecomment-2957849139
Thank you. > * `./gradlew eclipse`, `./gradlew test`, `./gradlew tidy`, etc seem to work correctly for me. Yup, I did try to check everything as I went along (regenerate, etc.). > * properties set in `~/.gradle/gradle.properties` are still respected, at least as far as passing tmpfs to tests. Did not see this in your list, so I checked it. "Alternatively, the generated gradle.properties is also supported since it declares project properties (equivalent to -Pkey=value)." gradle.properties just picks up project properties - it's an equivalent of saying -Pkey=value on command line. So yes, these will be respected just fine. I had some hopes we'd be able to make gradle.properties versioned, eventually, but I haven't found a way around setting the number of worker jvms. It's fine any way you set it. > * `:spotlessGradleScripts` seems incredibly slow, but it is just a one-time pain, the very first time you run it. I think it downloads. Yes, this one is a pain because the formatter is Eclipse-based and downloads half the internet... Again - haven't found a way to work around it but I love the fact i don't need to care about formatting groovy code and it takes care of it for me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
