Skip to content

Conversation

@Evleaps
Copy link
Contributor

@Evleaps Evleaps commented Oct 23, 2022

Just a small refactoring.
Because I was a problem with understanding and debugging the shouldInclude method.

Now is easier to understand and debug.

@codecov
Copy link

codecov bot commented Oct 23, 2022

Codecov Report

Merging #173 (e72a303) into main (f041be2) will increase coverage by 0.09%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##               main     #173      +/-   ##
============================================
+ Coverage     50.37%   50.47%   +0.09%     
  Complexity       66       66              
============================================
  Files            13       13              
  Lines           532      531       -1     
  Branches         99       97       -2     
============================================
  Hits            268      268              
  Misses          237      237              
+ Partials         27       26       -1     
Impacted Files Coverage Δ
...x/affectedmoduledetector/AffectedModuleDetector.kt 42.28% <71.42%> (+0.24%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@joshafeinberg joshafeinberg merged commit 952a017 into dropbox:main Oct 31, 2022
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.

2 participants