gradle: re-enable spotbugs everywhere, print output to console

[postamar]

It turns out that disabling spotbugs in the gradle.yml CI workflow interferes with the java CDK publish workflow. Since we've committed to continue to use spotbugs (it turns out that it is useful on occasion) this PR re-enables it.

This PR also improves the useability of spotbugs by printing its findings to the console instead of hiding them away in an inaccessible report file. I checked and the console output is perfectly usable:

> Task :airbyte-cdk:java:airbyte-cdk:db-sources:spotbugsTestFixtures
H D NP: checkStatusMap must be non-null but is marked as nullable  At AbstractSourcePerformanceTest.kt:[lines 85-100]
H C NP: io.airbyte.cdk.integrations.debezium.CdcSourceTest.getTestdb() may return null, but is declared @Nonnull  At CdcSourceTest.kt:[line 28]
H D NP: m must be non-null but is marked as nullable  At AbstractSourceDatabaseTypeTest.kt:[line 151]
H C NP: io.airbyte.cdk.integrations.standardtest.source.PythonSourceAcceptanceTest$Companion.getIMAGE_NAME() may return null, but is declared @Nonnull  At PythonSourceAcceptanceTest.kt:[line 158]
H D NP: m must be non-null but is marked as nullable  At SourceAcceptanceTest.kt:[line 452]
H D NP: obj must be non-null but is marked as nullable  At SourceAcceptanceTest.kt:[line 453]
H D NP: obj must be non-null but is marked as nullable  At SourceAcceptanceTest.kt:[line 252]
H D NP: m must be non-null but is marked as nullable  At SourceAcceptanceTest.kt:[line 251]
H D NP: messages must be non-null but is marked as nullable  At SourceAcceptanceTest.kt:[line 450]
H D NP: schemaName must be non-null but is marked as nullable  At JdbcSourceAcceptanceTest.kt:[line 318]
H C NP: io.airbyte.cdk.integrations.source.jdbc.test.JdbcSourceAcceptanceTest.getTestdb() may return null, but is declared @Nonnull  At JdbcSourceAcceptanceTest.kt:[line 42]

FYI @alafanechere this will cause a slight CI cost increase.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	⬜️ Ignored (Inspect)	Visit Preview		Apr 4, 2024 0:11am

[stephane-airbyte]

can we keep the html? I found it useful when fixing those

[postamar]

I've struggled to find a way to also print the console output, in addition to the HTML output. I'll try again next week.