Skip to content
This repository has been archived by the owner on Jul 19, 2019. It is now read-only.

Update javassist to 3.22.0-GA and GWT mockito to 1.1.7 #68

Merged
merged 2 commits into from
Nov 6, 2017

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Oct 27, 2017

CQs

What does this PR do?

Update javassist to a version compliant with Java9
Also update GWT mockito to 1.1.7 compliant with javassit 3.22

What issues does this PR fix or reference?

eclipse-che/che#5326

Previous behavior

Some failures when using it with Java9

New behavior

Fix failures with Java9

Tests written?

No

Docs updated?

No

@benoitf benoitf changed the title Update javassist to 3.22.0-GA [WiP] Update javassist to 3.22.0-GA and GWT mockito to 1.1.7 Oct 27, 2017
Copy link
Contributor

@skabashnyuk skabashnyuk left a comment

Choose a reason for hiding this comment

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

Please contact @vparfonov to know if he is ok too.

@benoitf
Copy link
Contributor Author

benoitf commented Oct 27, 2017

yes sure, BTW I've provided PR for GWT Mockito project as well so I'm waiting some feedback as well.

@benoitf benoitf requested a review from vparfonov October 27, 2017 15:07
@benoitf benoitf changed the title [WiP] Update javassist to 3.22.0-GA and GWT mockito to 1.1.7 Update javassist to 3.22.0-GA and GWT mockito to 1.1.7 Nov 2, 2017
@benoitf
Copy link
Contributor Author

benoitf commented Nov 2, 2017

CQs accepted

@benoitf
Copy link
Contributor Author

benoitf commented Nov 3, 2017

I will merge this PR on monday

@benoitf benoitf merged commit d004883 into che6 Nov 6, 2017
@benoitf benoitf deleted the update-javassist branch November 6, 2017 06:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants