Closed markvr closed 1 year ago
hi, does this look ok? Can it be merged? :)
Thanks!
One thing I noted in testing is that this (and so presumably the Gradle support) works without needing the "Enable experimental features" checkbox enabled.
I wish I recalled why I added that... Maybe it's time to drop that option. 🤔
Have essentially copy and pasted from existing Gradle support.
Did try to pull common code into an abstract
ExecutionEnvironmentProvider
class but becauseMavenExecutionEnvironmentProvider
andGradleExecutionEnvironmentProvider
don't inherit from a common superclass it just ended up being messy and worse than just having the classes separate.One thing I noted in testing is that this (and so presumably the Gradle support) works without needing the "Enable experimental features" checkbox enabled.