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

Increase jenkins version #2

Merged
merged 1 commit into from
Apr 16, 2020
Merged

Increase jenkins version #2

merged 1 commit into from
Apr 16, 2020

Conversation

timja
Copy link
Member

@timja timja commented Apr 16, 2020

Bump jenkins version up so there's no longer an implied dependency on trilead-api

ref: #1 (comment)

Bump jenkins version up so there's no longer an implied dependency on trilead-api
@timja timja requested review from jtnord, amuniz, jetersen and a team April 16, 2020 10:11
@timja timja added the chore label Apr 16, 2020
@timja timja merged commit d6e5e9b into master Apr 16, 2020
@timja timja deleted the timja-patch-1 branch April 16, 2020 11:15
@timja
Copy link
Member Author

timja commented Apr 16, 2020

🤦 release PR isn't merged yet, so release just failed

@jetersen
Copy link
Member

once merged you should be able to resume release.

@timja
Copy link
Member Author

timja commented Apr 16, 2020

k

@jetersen
Copy link
Member

by running the same mvn release:prepare release:perform

@jtnord
Copy link
Member

jtnord commented Apr 16, 2020

Thanks again @timja

@@ -29,7 +29,7 @@
<maven.javadoc.skip>true</maven.javadoc.skip>
<snakeyaml.version>1.26</snakeyaml.version>
<java.level>8</java.level>
<jenkins.version>2.164.3</jenkins.version>
<jenkins.version>2.204.6</jenkins.version>
Copy link
Member

Choose a reason for hiding this comment

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

2.190.1 would have done it, no?

Copy link
Member Author

Choose a reason for hiding this comment

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

Most plugins seem to be taking 2.204 no real reason though for this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants