This is an automated email from the ASF dual-hosted git repository.
johnyangk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nemo.git
The following commit(s) were added to refs/heads/master by this push:
new eea3f57 [NEMO-248] Validate javadoc in mvn phase (#139)
eea3f57 is described below
commit eea3f57e3e19a7a9cce368fd6102bbaf403a8315
Author: Taegeon Um <[email protected]>
AuthorDate: Wed Oct 31 10:10:45 2018 +0900
[NEMO-248] Validate javadoc in mvn phase (#139)
JIRA: [NEMO-248: Validate javadoc in mvn
phase](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-248)
**Major changes:**
- Add javadoc validation to validate phase
- Remove mvn javdoc:javdoc in .travis.yml
**Minor changes to note:**
-
**Tests for the changes:**
-
**Other comments:**
-
Closes #139
---
.travis.yml | 5 ++---
pom.xml | 7 +++++++
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 413b2ac..72fbd98 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -26,9 +26,8 @@ jdk:
- oraclejdk8
script:
# the following command line builds the project, runs the tests with
coverage and then execute the SonarCloud analysis
- - if [ "$TRAVIS_PULL_REQUEST" == false ]; then travis_retry mvn clean
org.jacoco:jacoco-maven-plugin:prepare-agent verify sonar:sonar -B -q -ff
-Dsurefire.useFile=false -Dorg.slf4j.simpleLogger.defaultLogLevel=info; fi
- - if [ "$TRAVIS_PULL_REQUEST" != false ]; then travis_retry mvn clean verify
-B -q -ff -Dsurefire.useFile=false
-Dorg.slf4j.simpleLogger.defaultLogLevel=info; fi
- - mvn clean install -DskipTests && mvn javadoc:javadoc
+ - if [ "$TRAVIS_PULL_REQUEST" == false ]; then travis_retry mvn clean
validate; mvn clean org.jacoco:jacoco-maven-plugin:prepare-agent verify
sonar:sonar -B -q -ff -Dsurefire.useFile=false
-Dorg.slf4j.simpleLogger.defaultLogLevel=info; fi
+ - if [ "$TRAVIS_PULL_REQUEST" != false ]; then travis_retry mvn clean
validate; mvn clean verify -B -q -ff -Dsurefire.useFile=false
-Dorg.slf4j.simpleLogger.defaultLogLevel=info; fi
notifications:
slack:
diff --git a/pom.xml b/pom.xml
index 9d7e769..1675e9c 100644
--- a/pom.xml
+++ b/pom.xml
@@ -247,6 +247,13 @@ under the License.
</goals>
<phase>site</phase>
</execution>
+ <execution>
+ <id>test-javadoc</id>
+ <goals>
+ <goal>javadoc</goal>
+ </goals>
+ <phase>validate</phase>
+ </execution>
</executions>
</plugin>
<plugin>
