intellij-platform-gradle-plugin icon indicating copy to clipboard operation
intellij-platform-gradle-plugin copied to clipboard

verifyPlugin does not fails builds if verificationReportsFormats does not include PLAIN reports

Open sbouchet opened this issue 10 months ago • 1 comments

What happened?

we're using pluginverification task and noticed that even the report mention errors, the task don't fail if we specify the correct failure level, if the PLAIN verificationFormat is missing it fails only if the ALL level is used

Relevant log output or stack trace

we're currently using

pluginVerification {
        failureLevel = listOf(INVALID_PLUGIN, COMPATIBILITY_PROBLEMS, MISSING_DEPENDENCIES)
       verificationReportsFormats = listOf(MARKDOWN)

and build outuput is


...
BUILD SUCCESSFUL in 4m 46s

but as soon as i change to

pluginVerification {
        failureLevel = ALL
        verificationReportsFormats = listOf(MARKDOWN)

the same build fails with

> Task :verifyPlugin FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':verifyPlugin'.
> SCHEDULED_FOR_REMOVAL_API_USAGES: Plugin uses API marked as scheduled for removal (ApiStatus.@ScheduledForRemoval). Check Plugin Verifier report for more details.

that is the correct expected output

Steps to reproduce

check out https://github.com/redhat-developer/intellij-kubernetes/commit/5eea7ffb79349a7c268525abbf170f9b7d7df1fa ./gradlew verifyPlugin will not fail change build.gradle.kts to use ALL as failureLevel and it will fail or change verificationReportFormats to 'verificationReportsFormats = listOf(MARKDOWN, PLAIN)'

Gradle IntelliJ Plugin version

2.2.1

Gradle version

8.5

Operating System

Linux

Link to build, i.e. failing GitHub Action job

https://github.com/redhat-developer/intellij-kubernetes/pull/824

sbouchet avatar Jan 31 '25 08:01 sbouchet

found out it's because we're not using PLAIN report format. still an issue though, but workaround is to always set PLAIN format

sbouchet avatar Feb 05 '25 14:02 sbouchet