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

Removed JDK7 build from CI to fix Travis regression with Jdk7 #1184

Merged
merged 2 commits into from Sep 4, 2017

Conversation

mockitoguy
Copy link
Member

@mockitoguy mockitoguy commented Sep 4, 2017

Summary:

  • Removed our JDK7 build from Travis CI completely
  • Fixed issues with Mockito Travis CI build

Reference:

Details:

  • I have tried to get the build working with jdk7 but I couldn't get it to work (example failure). I kept getting:

Exception in thread "main" javax.net.ssl.SSLException: java.security.ProviderException.

We remember this issue from our Jdk6 build, we could not fix it and eventually we have stopped building with Jdk6.

  • We still publish with Java6 compatibility and we have animal sniffer set up to catch regressions. It should be ok to build only on Jdk8.
  • Jdk6 and jdk7 has been deprecated years ago, we need to move on.

- I have tried to get the build working with jdk7 but I couldn't get it to work. I kept getting: Exception in thread "main" javax.net.ssl.SSLException: java.security.ProviderException. We remember this issue from our Jdk6 build, we could not fix it and eventually we have stopped building with Jdk6.
- We still publish with Java6 compatibility and we have animal sniffer set up to catch regressions. It should be ok to build only on Jdk8.
- Jdk6 and jdk7 has been deprecated years ago, we need to move on.
@codecov-io
Copy link

codecov-io commented Sep 4, 2017

Codecov Report

Merging #1184 into release/2.x will decrease coverage by 0.25%.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                @@
##             release/2.x    #1184      +/-   ##
=================================================
- Coverage          86.93%   86.67%   -0.26%     
- Complexity          2299     2309      +10     
=================================================
  Files                287      287              
  Lines               5825     5900      +75     
  Branches             686      703      +17     
=================================================
+ Hits                5064     5114      +50     
- Misses               564      585      +21     
- Partials             197      201       +4
Impacted Files Coverage Δ Complexity Δ
.../internal/creation/bytebuddy/MockMethodAdvice.java 65.4% <0%> (+0.85%) 27% <0%> (+10%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c61f1a...da3f20e. Read the comment docs.

@raphw raphw merged commit 500b577 into release/2.x Sep 4, 2017
@mockitoguy mockitoguy changed the title Travis Removed JDK7 build from CI to fix Travis regression with Jdk7 Sep 4, 2017
@TimvdLippe TimvdLippe deleted the travis branch October 22, 2017 14:15
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.

None yet

3 participants