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

Replace Reflections with FastClassPathScanner (#2243) #2346

Conversation

PascalSchumacher
Copy link
Contributor

No description provided.

@PascalSchumacher PascalSchumacher force-pushed the replace_reflections_with_fast_classpath_scanner branch 7 times, most recently from 2df7a9d to 35fadb1 Compare April 7, 2018 09:36
@codecov
Copy link

codecov bot commented Apr 7, 2018

Codecov Report

Merging #2346 into master will increase coverage by 0.34%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2346      +/-   ##
============================================
+ Coverage     95.07%   95.41%   +0.34%     
+ Complexity     2979     2956      -23     
============================================
  Files           339      336       -3     
  Lines          7917     7858      -59     
  Branches        642      620      -22     
============================================
- Hits           7527     7498      -29     
+ Misses          249      227      -22     
+ Partials        141      133       -8

@PascalSchumacher PascalSchumacher force-pushed the replace_reflections_with_fast_classpath_scanner branch from 35fadb1 to 3674dd1 Compare April 7, 2018 09:59
@dilipkrish
Copy link
Member

@PascalSchumacher thanks for the PR. Will hold of on pulling this in until we move to 3.0 (breaking change to move to 1.8 jdk)

@dilipkrish dilipkrish added the PR label Apr 8, 2018
@PascalSchumacher PascalSchumacher force-pushed the replace_reflections_with_fast_classpath_scanner branch 4 times, most recently from 2835801 to f952d54 Compare May 8, 2018 19:14
@PascalSchumacher PascalSchumacher force-pushed the replace_reflections_with_fast_classpath_scanner branch from f952d54 to 7cdbff7 Compare June 16, 2018 11:55
@dilipkrish dilipkrish merged commit 7cdbff7 into springfox:master Jun 30, 2018
@dilipkrish dilipkrish added this to the 3.0 milestone Jun 30, 2018
@dilipkrish
Copy link
Member

Thank you for the PR! Its been merged!

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.

None yet

2 participants