Closed romeara closed 4 years ago
Merging #22 into master will increase coverage by
0.38%
. The diff coverage is90.24%
.
@@ Coverage Diff @@
## master #22 +/- ##
============================================
+ Coverage 84.65% 85.04% +0.38%
- Complexity 240 252 +12
============================================
Files 24 25 +1
Lines 717 749 +32
Branches 47 53 +6
============================================
+ Hits 607 637 +30
Misses 91 91
- Partials 19 21 +2
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
...rchartlabs/flare/plugins/model/ConstraintFile.java | 100% <100%> (ø) |
13 <3> (+1) |
:arrow_up: |
...abs/flare/plugins/model/DependencyConstraints.java | 92.85% <75%> (ø) |
10 <1> (ø) |
:arrow_down: |
.../starchartlabs/flare/plugins/model/Constraint.java | 92.59% <92.59%> (ø) |
11 <11> (?) |
Merging #22 into master will not change security risk.
Previously, constraints from a dependency constraints file were unconditionally applied to all configurations. This could lead to test-only constraints leaking into published documents, such as the dependencyManagement section of Maven POMs.
This change adds the ability to restrict the configuration(s) a constraint is applied to from within the dependency constraint file, which allows clients to determine how broadly they wish each constraint to be applied