Skip to content

Commit 62779ef

Browse files
authored
chore: fix duplicate plugin warning and disable snapshots (#378)
* Don't use snapshots anymore since we don't publish them and we don't use them for 'integrate at HEAD' development * Consolidate the duplicate definition of the antrun plugin to eliminate maven warning <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated release configuration settings across main and release branch configurations. * Reorganized Maven build profile configuration to improve build process consistency. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 9b38f82 commit 62779ef

3 files changed

Lines changed: 23 additions & 30 deletions

File tree

.github/release-please/release-please-config.main.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
"separate-pull-requests": false,
66
"include-component-in-tag": false,
77
"group-pull-request-title-pattern": "chore(${branch}): release ${version}",
8+
"skip-snapshot": true,
89
"packages": {
910
".": {}
1011
},

.github/release-please/release-please-config.release_branches.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
"separate-pull-requests": false,
66
"include-component-in-tag": false,
77
"group-pull-request-title-pattern": "chore(${branch}): release ${version}",
8+
"skip-snapshot": true,
89
"packages": {
910
".": {}
1011
},

sdk/pom.xml

Lines changed: 21 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -545,7 +545,7 @@
545545
</properties>
546546
<build>
547547
<plugins>
548-
<!-- Plugin to download and unpack the Jazzer agent -->
548+
<!-- Plugin to download/unpack Jazzer agent and execute fuzzing -->
549549
<plugin>
550550
<groupId>org.apache.maven.plugins</groupId>
551551
<artifactId>maven-antrun-plugin</artifactId>
@@ -575,35 +575,6 @@
575575
<goal>run</goal>
576576
</goals>
577577
</execution>
578-
</executions>
579-
</plugin>
580-
581-
<!-- Copy dependencies to a directory for Jazzer to reference -->
582-
<plugin>
583-
<groupId>org.apache.maven.plugins</groupId>
584-
<artifactId>maven-dependency-plugin</artifactId>
585-
<version>3.4.0</version>
586-
<executions>
587-
<execution>
588-
<id>copy-dependencies</id>
589-
<phase>process-test-classes</phase>
590-
<goals>
591-
<goal>copy-dependencies</goal>
592-
</goals>
593-
<configuration>
594-
<outputDirectory>${project.build.directory}/dependency-jars</outputDirectory>
595-
<includeScope>test</includeScope>
596-
</configuration>
597-
</execution>
598-
</executions>
599-
</plugin>
600-
601-
<!-- Plugin to execute Jazzer -->
602-
<plugin>
603-
<groupId>org.apache.maven.plugins</groupId>
604-
<artifactId>maven-antrun-plugin</artifactId>
605-
<version>3.1.0</version>
606-
<executions>
607578
<execution>
608579
<id>run-jazzer-fuzzing</id>
609580
<phase>verify</phase>
@@ -635,6 +606,26 @@
635606
</execution>
636607
</executions>
637608
</plugin>
609+
610+
<!-- Copy dependencies to a directory for Jazzer to reference -->
611+
<plugin>
612+
<groupId>org.apache.maven.plugins</groupId>
613+
<artifactId>maven-dependency-plugin</artifactId>
614+
<version>3.4.0</version>
615+
<executions>
616+
<execution>
617+
<id>copy-dependencies</id>
618+
<phase>process-test-classes</phase>
619+
<goals>
620+
<goal>copy-dependencies</goal>
621+
</goals>
622+
<configuration>
623+
<outputDirectory>${project.build.directory}/dependency-jars</outputDirectory>
624+
<includeScope>test</includeScope>
625+
</configuration>
626+
</execution>
627+
</executions>
628+
</plugin>
638629
</plugins>
639630
</build>
640631
</profile>

0 commit comments

Comments
 (0)