Skip to content

Conversation

@chris-mitchell
Copy link
Contributor

The module detector was being configured mid-way through the project graph evaluation. This caused some project relations to not be identified and projects to be missed.

This pushes the plugins call until after the projects have been evaluated, and implements a check to ensure everything has been evaluated before proceeding.

Copy link
Contributor

@digitalbuddha digitalbuddha left a comment

Choose a reason for hiding this comment

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

fuuun

@digitalbuddha
Copy link
Contributor

closes #42

@codecov
Copy link

codecov bot commented Dec 11, 2020

Codecov Report

Merging #43 (bdf006c) into main (331a1d7) will increase coverage by 2.60%.
The diff coverage is 6.66%.

Impacted file tree graph

@@ Coverage Diff @@ ## main #43 +/- ## ============================================ + Coverage 39.04% 41.64% +2.60%  - Complexity 25 29 +4  ============================================ Files 8 8 Lines 397 401 +4 Branches 87 89 +2 ============================================ + Hits 155 167 +12  + Misses 211 203 -8  Partials 31 31 
Impacted Files Coverage Δ Complexity Δ
...x/affectedmoduledetector/AffectedModuleDetector.kt 38.88% <0.00%> (-1.12%) 1.00 <0.00> (ø)
...ctedmoduledetector/AffectedModuleDetectorPlugin.kt 21.81% <10.00%> (+21.81%) 4.00 <0.00> (+4.00)

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 286100a...bdf006c. Read the comment docs.

@chris-mitchell chris-mitchell merged commit 3fec06b into dropbox:main Dec 11, 2020
@chris-mitchell chris-mitchell deleted the fixConfigurationPhases branch December 11, 2020 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants