Skip to content

update parent pom version#72

Merged
undera merged 3 commits intojenkinsci:masterfrom
rpionke:master
Oct 27, 2016
Merged

update parent pom version#72
undera merged 3 commits intojenkinsci:masterfrom
rpionke:master

Conversation

@rpionke
Copy link
Copy Markdown
Member

@rpionke rpionke commented Oct 25, 2016

@undera
Copy link
Copy Markdown
Member

undera commented Oct 26, 2016

Thanks for helping with this.
Travis CI build is still failing. Can you fix it?

@codecov-io
Copy link
Copy Markdown

Current coverage is 45.59% (diff: 0.00%)

Merging #72 into master will decrease coverage by 1.97%

@@             master        #72   diff @@
==========================================
  Files            36         29     -7   
  Lines          3000       2485   -515   
  Methods           0          0          
  Messages          0          0          
  Branches        489        404    -85   
==========================================
- Hits           1427       1133   -294   
+ Misses         1455       1233   -222   
- Partials        118        119     +1   

Powered by Codecov. Last update b8b6e98...9c637fc

@rpionke
Copy link
Copy Markdown
Member Author

rpionke commented Oct 26, 2016

fixed Travis CI build by increasing jvm args -XX:MaxPermSize=256m -Xmx1024m see travis-ci/travis-ci#5597

@undera
Copy link
Copy Markdown
Member

undera commented Oct 27, 2016

Many-many thanks! You helped a lot!

@undera undera merged commit 40ab8fe into jenkinsci:master Oct 27, 2016
Comment thread pom.xml
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
<configuration>
<additionalparam>-Xdoclint:none</additionalparam>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not do this. Rather fix any actual Javadoc errors. Typically there are not many and it only takes a couple of minutes. warnings can be disregarded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants