Add environmentVariables to gradle task quarkusDev

[Eng-Fouad]

Closes #32661

[quarkus-bot]

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

• title should preferably start with an uppercase character (if it makes sense!)

This message is automatically generated by a bot.

[aloubyansky]

Thanks @Eng-Fouad, do you think you could add a little test in https://github.com/quarkusio/quarkus/tree/main/integration-tests/gradle for this feature?

[Eng-Fouad]

Thanks @Eng-Fouad, do you think you could add a little test in https://github.com/quarkusio/quarkus/tree/main/integration-tests/gradle for this feature?

Yes sure, I will do.

[Eng-Fouad]

@aloubyansky I just added a simple test. However, I could not run the Quarkus project on my machine, so I wrote the test without compilation. Could you please verify the test from your side? Thanks.

[aloubyansky]

Thanks @Eng-Fouad, let's see what the CI says.

[aloubyansky]

Looks like it's passing @Eng-Fouad, thanks! Would you mind also adding a note about it in https://github.com/quarkusio/quarkus/blob/main/docs/src/main/asciidoc/gradle-tooling.adoc? It'll end up in https://quarkus.io/guides/gradle-tooling#dev-mode. You could do that in a separate PR, if you like.

[Eng-Fouad]

Looks like it's passing @Eng-Fouad, thanks! Would you mind also adding a note about it in https://github.com/quarkusio/quarkus/blob/main/docs/src/main/asciidoc/gradle-tooling.adoc? It'll end up in https://quarkus.io/guides/gradle-tooling#dev-mode. You could do that in a separate PR, if you like.

OK, I will add it to docs 👍

[Eng-Fouad]

@aloubyansky using Kotlin DSL, which one is correct?

tasks.quarkusDev {
    environmentVariables = mapOf("FOO_VALUE" to "abc", "BAR_VALUE" to "def")
}

or

tasks.quarkusDev {
    environmentVariables.set(mapOf("FOO_VALUE" to "abc", "BAR_VALUE" to "def"))
}

[Eng-Fouad]

I chose: environmentVariables.set()

[gastaldi]

Nice!

[github-actions]

🙈 The PR is closed and the preview is expired.

[quarkus-bot]

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

[maxandersen]

does this not overwrite users actual environment meaning users with FOO explicitly set in build.gradle can no longer do:

FOO=whatever gradle quarkusDev

as gradle build will override the environment?

That is not good IMO as it changes the order of priority/locality.