Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-33321] Upgrade to new parent pom #22

Merged
merged 3 commits into from
Jul 2, 2016

Conversation

armfergom
Copy link
Contributor

Upgrade to new parent pom. Includes many improvements. More info here

  • Upgrade to new parent pom (version 2.3) keeping the same baseline version (1.625).
  • Fix Injected test due to upgrade.
  • Fix Findbugs issues.
  • Fix Javadoc so that maven javadoc plugin doesn't complain.

@reviewbybees

@jenkinsadmin
Copy link
Member

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

@armfergom armfergom changed the title [WiP] [JENKINS-33321] Upgrade to new parent pom [JENKINS-33321] Upgrade to new parent pom Mar 4, 2016
@ghost
Copy link

ghost commented Mar 4, 2016

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

<properties>
<jenkins.version>1.625</jenkins.version>
<java.level>6</java.level>
</properties>
Copy link

Choose a reason for hiding this comment

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

Copy link

Choose a reason for hiding this comment

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

Use the profiles at the end of the parent POM as an orientation of the java levels to use (they are LTS boundaries, not exact version in which it change, but serve as an quick guide).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I checked the version to use in org.jenkins-ci.jenkins
But I didn't notice it was overriden by org.jenkins-ci.main.pom

My bad. Updated now.

@andresrc
Copy link

andresrc commented Mar 4, 2016

🐝

1 similar comment
@recena
Copy link

recena commented Mar 4, 2016

🐝

@armfergom
Copy link
Contributor Author

@reviewbybees done

@ghost
Copy link

ghost commented Mar 7, 2016

This pull request has completed our internal processes and we now respectfully request the maintainers of this repository to consider our proposal contained within this pull request for merging.

@armfergom
Copy link
Contributor Author

In order to make the build of the plugin stable using baseline 2.0-rc-1, the parent pom to use must be 2.4+ where version 2.6 of the Jenkins Test Harness is picked up.

Added one commit bumping parent pom from 2.3 to 2.6.

@reviewbybees esp. @recena @andresrc who initially reviewed the PR

@andresrc
Copy link

🐝

1 similar comment
@recena
Copy link

recena commented Apr 18, 2016

🐝

@armfergom
Copy link
Contributor Author

@reviewbybees done

@armfergom
Copy link
Contributor Author

Hi @gboissinot, is this PR something that may be considered to be merged? :) Thanks!

@recena
Copy link

recena commented Jun 23, 2016

@armfergom I recommend you to send an email to jenkins-dev mailing list.

@marshall777 marshall777 merged commit f60ca3c into jenkinsci:master Jul 2, 2016
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.

5 participants