merlimat commented on a change in pull request #7549:
URL: https://github.com/apache/pulsar/pull/7549#discussion_r455460448
##########
File path: .github/workflows/ci-cpp.yaml
##########
@@ -70,9 +70,9 @@ jobs:
docker rmi $(docker images -q) -f
df -h
- - name: build pacakge
+ - name: build package
if: steps.docs.outputs.changed_only == 'no'
- run: mvn -B -Dorg.slf4j.simpleLogger.defaultLogLevel=ERROR package
-DskipTests
+ run: mvn -B -ntp -Dorg.slf4j.simpleLogger.defaultLogLevel=ERROR -ntp
install -Pcore-modules -DskipTests
Review comment:
Yes, but unit tests are already included in broker module, there's no
need to compile the entire project, just the core modules
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
