Avoid storing timestamp in Gradle.properties

[gastaldi]

• Fixes Running Quarkus CLI rewrites gradle.properties #41182

[gsmet]

Does it also avoid rewriting the file and respect the formatting of the existing file? Because I think that's what the original issue was about.

[gastaldi]

@gsmet yes, I tested locally and the order is preserved. In another words, running the steps from the original issue does not change the properties file

[gastaldi]

Using StringWriter because of codejive/java-properties#23

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 423b4f4.

✅ 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.

You can consult the Develocity build scans.

---

Flaky tests - Develocity

⚙️ JVM Tests - JDK 21

📦 independent-projects/resteasy-reactive/server/vertx

✖ org.jboss.resteasy.reactive.server.vertx.test.sse.SseServerTestCase.shouldNotTryToSendToClosedSink - History

• 1 expectation failed. Response body doesn't match expectation. Expected: "true" Actual: false - java.lang.AssertionError

java.lang.AssertionError: 
1 expectation failed.
Response body doesn't match expectation.
Expected: "true"
  Actual: false

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)

---

⚙️ Native Tests - Messaging1

📦 integration-tests/reactive-messaging-kafka

✖ io.quarkus.it.kafka.KafkaConnectorIT.testRequestReply - History

• iterable contents differ at index [3], expected: <reply-4> but was: <{"details":"Error id 3816f35d-6178-4e7c-843d-27cbfb52fb46-1","stack":""}> - org.opentest4j.AssertionFailedError

org.opentest4j.AssertionFailedError: iterable contents differ at index [3], expected: <reply-4> but was: <{"details":"Error id 3816f35d-6178-4e7c-843d-27cbfb52fb46-1","stack":""}>
	at io.quarkus.it.kafka.KafkaConnectorTest.testRequestReply(KafkaConnectorTest.java:111)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at io.quarkus.test.junit.QuarkusTestExtension.interceptTestMethod(QuarkusTestExtension.java:812)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

[gsmet]

Looks like a nice improvement. I will merge.

Could you have a look at what @radcortez did in the quarkus config CLI? Maybe it would benefit from this too.

[maxandersen]

awesome stuff @gastaldi !