Skip to content

Commit

Permalink
Merge pull request #11101 from murdos/jqassistant-verify-phase
Browse files Browse the repository at this point in the history
fix(jqassistant): execute maven plugin on verify phase
  • Loading branch information
murdos authored Oct 12, 2024
2 parents 6046fab + cb63046 commit 64713e9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ private static MavenPluginOptionalBuilder jQAssistantPluginBuilder() {
private static MavenPlugin jQAssistantPluginManagement() {
return jQAssistantPluginBuilder()
.versionSlug("jqassistant")
.addExecution(pluginExecution().goals("scan", "analyze").id("default-cli"))
.addExecution(pluginExecution().goals("scan", "analyze").phase(VERIFY).id("default-cli"))
.build();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ void shouldBuildModule() {
<executions>
<execution>
<id>default-cli</id>
<phase>verify</phase>
<goals>
<goal>scan</goal>
<goal>analyze</goal>
Expand Down

0 comments on commit 64713e9

Please sign in to comment.