Remove @NotNull annotations

[geoand]

This are sometimes added by IntelliJ but they should not be part of the Quarkus codebase

[famod]

Couldn't we exclude & ban the jar it's coming from?

[gastaldi]

Couldn't we exclude & ban the jar it's coming from?

Not a bad idea, but I think this is not included in the pom.xml, but in the IDEA project directly

[famod]

It must be in some jar on the classpath, otherwise the compiler plugin would complain, no?

[geoand]

We should definitely ban them, I haven't looked into that

[famod]

It seems to be coming from org/jetbrains/annotations/24.0.1/annotations-24.0.1.jar.

I think it might be impossible to get rid of it entirely, so that forbidden-apis thingy might be the way to go.

[quarkus-bot]

Failing Jobs - Building 5954021

Status	Name	Step	Failures	Logs	Raw logs
✔️	Gradle Tests - JDK 11
✖	Gradle Tests - JDK 11 Windows	Build	Failures	Logs	Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 Windows #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

✖ io.quarkus.gradle.devmode.KotlinProjectWithCompilerArgsDevModeTest.main line 17 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.test.devmode.util.DevModeTestUtils that uses java.util.function.Supplier, java.util.function.Supplierjava.util.concurrent.atomic.AtomicReference, java.util.concurrent.atomic.AtomicReferencejava.lang.String, java.lang.Stringboolean was not fulfilled within 2 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)