Adds [App]ArtifactKey.toGacString() method to strip the type from the string

[aloubyansky]

Switching from JsonInclude.Include.NON_EMPTY to JsonInclude.Include.NON_DEFAULT also gets rid of the empty arrays in the json (but once we upgrade to the next version of the quarkus-platform-bom-maven-plugin).

[gastaldi]

LGTM

[quarkus-bot]

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building a8d13c2

Status	Name	Step	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 11	Build	Test failures	Logs	Raw logs
✖	JVM Tests - JDK 11 Windows	Build	Test failures	Logs	Raw logs
✔️	JVM Tests - JDK 16

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 devtools/cli

✖ io.quarkus.cli.CliProjectGradleTest.testCreateAppOverrides - More details - Source on GitHub

---

⚙️ JVM Tests - JDK 11 Windows #

📦 devtools/cli

✖ io.quarkus.cli.CliProjectGradleTest.testCreateAppOverrides - More details - Source on GitHub

[aloubyansky]

This has revealed a bug in our Gradle plugin related to dependency handling.

[aloubyansky]

@glefloch fyi, the second commit is supposed to fix the parsing of dependency conditions and checking whether they are satisfied.

[quarkus-bot]

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building 4d7b362

Status	Name	Step	Test failures	Logs	Raw logs
✔️	JVM Tests - JDK 11
✖	JVM Tests - JDK 11 Windows	Build	Test failures	Logs	Raw logs
✔️	JVM Tests - JDK 16

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 Windows #

📦 integration-tests/resteasy-reactive-rest-client

✖ io.quarkus.it.rest.client.BasicTest.shouldMakeJsonRequest line 40 - More details - Source on GitHub

[gsmet]

I only backported the first commit, the second commit is not backportable as the feature is not in 2.0.