Skip to content

Issue 884: Upgrade powermock and mockito to support java 9#885

Closed
sijie wants to merge 1 commit into
apache:masterfrom
sijie:use_powermock2
Closed

Issue 884: Upgrade powermock and mockito to support java 9#885
sijie wants to merge 1 commit into
apache:masterfrom
sijie:use_powermock2

Conversation

@sijie
Copy link
Copy Markdown
Member

@sijie sijie commented Dec 19, 2017

Descriptions of the changes in this PR:

powermock supports java9 at the new version (see powermock/powermock#783). this pull request upgrades the powermock and mockito versions, so we are able to run tests on java 9.

Copy link
Copy Markdown
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

+1 lgtm

Copy link
Copy Markdown
Member

@jiazhai jiazhai left a comment

Choose a reason for hiding this comment

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

+1

@jiazhai jiazhai closed this in 67d560f Dec 19, 2017
ArvinDevel pushed a commit to ArvinDevel/bookkeeper that referenced this pull request Dec 19, 2017
Descriptions of the changes in this PR:

powermock supports java9 at the new version (see powermock/powermock#783). this pull request upgrades the powermock and mockito versions, so we are able to run tests on java 9.

Author: Sijie Guo <sijie@apache.org>

Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <None>

This closes apache#885 from sijie/use_powermock2, closes apache#884
@sijie sijie deleted the use_powermock2 branch July 16, 2018 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants