Skip to content

Commit

Permalink
Merge pull request #1920 from akto-api-security/add-make-proto-mvn
Browse files Browse the repository at this point in the history
added protobuf generation in mvn
  • Loading branch information
ayushaga14 authored Jan 6, 2025
2 parents a44a716 + c04cfcd commit 4c9150c
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 3 deletions.
25 changes: 23 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,29 @@
<dataFile>${jacoco.reportPath}</dataFile>
<append>true</append>
</configuration>
</plugin>
</plugin>
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>exec-maven-plugin</artifactId>
<version>3.1.0</version>
<executions>
<execution>
<id>run-make-root-only</id>
<phase>compile</phase>
<goals>
<goal>exec</goal>
</goals>
<configuration>
<executable>make</executable>
<arguments>
<argument>proto-gen</argument>
</arguments>
<failOnError>true</failOnError>
</configuration>
<inherited>false</inherited>
</execution>
</executions>
</plugin>
</plugins>
</build>

Expand Down Expand Up @@ -126,4 +148,3 @@
</dependency>
</dependencies>
</project>

2 changes: 1 addition & 1 deletion scripts/proto-gen.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
# Please install buf if not already installed by following the instructions at https://docs.buf.build/installation
if ! command -v buf >/dev/null 2>&1; then
echo "Please install buf if not already installed by following the instructions at https://docs.buf.build/installation"
exit
exit 1
fi

buf lint protobuf
Expand Down

0 comments on commit 4c9150c

Please sign in to comment.