Conversation
|
Could you please also explain why the same cannot be achieved with scope / path excludes in combination with |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #12518 +/- ##
=========================================
Coverage 59.21% 59.21%
+ Complexity 1893 1891 -2
=========================================
Files 366 366
Lines 13838 13838
Branches 1462 1462
=========================================
Hits 8194 8194
Misses 5114 5114
Partials 530 530
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
4147f9f to
84fc7a4
Compare
I need to reach out to the customer to find what what their requirements again: I think they had some architecture specific packages that could not simply be excluded by scope / path excludes. |
84fc7a4 to
d050c61
Compare
Add a new `allowedDependenciesPath` configuration option that points to a file listing the dependencies (by `apparentName`) that are allowed to remain in the dependency tree. Any dependency not present in that file, together with its own subtree, is removed from the resolved `BazelModule` before further processing. This is useful to prune dependencies that are not relevant for the analysis, for example test-only or platform-specific dependencies that would otherwise pollute the dependency graph." Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
d050c61 to
57fdbbe
Compare
|
I have a hunch the use of this feature may be mostly by Bosch itself. |
Add a new
allowedDependenciesPathconfiguration option that points to a file listing the dependencies (byapparentName) that are allowed to remain in the dependency tree. Any dependency not present in that file, together with its own subtree, is removed from the resolvedBazelModulebefore further processing.This is useful to prune dependencies that are not relevant for the analysis, for example test-only or platform-specific dependencies that would otherwise pollute the dependency graph."
Note
The implementation in this PR was suggested in a ORT Community meeting.