-
Notifications
You must be signed in to change notification settings - Fork 23
feat: add enforce linkage checker #51
Conversation
Codecov Report
@@ Coverage Diff @@
## master #51 +/- ##
=========================================
Coverage 58.34% 58.34%
Complexity 117 117
=========================================
Files 18 18
Lines 1145 1145
Branches 3 3
=========================================
Hits 668 668
Misses 475 475
Partials 2 2Continue to review full report at Codecov.
|
|
We need to fix this before we can turn on this check: Dependency convergence error for com.google.errorprone:error_prone_annotations:2.3.3 paths to dependency are: [WARNING] Rule 2: org.apache.maven.plugins.enforcer.RequireUpperBoundDeps failed with message: |
I tried to resolve it but we have to wait until this pr gets merge grpc/grpc-java#6574 |
elharo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test failure looks related though since it's only on Java 8 maybe you just need to clear caches or some such:
[ERROR] Failed to execute goal on project proto-google-cloud-asset-v1: Could not resolve dependencies for project com.google.api.grpc:proto-google-cloud-asset-v1:jar:0.81.1-SNAPSHOT: Could not transfer artifact com.google.errorprone:error_prone_annotations:jar:2.3.4 from/to central (https://repo.maven.apache.org/maven2): /root/.m2/repository/com/google/errorprone/error_prone_annotations/2.3.4/error_prone_annotations-2.3.4.jar.part (No such file or directory) -> [Help 1]
[ERROR]
|
@elharo PTAL |
elharo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is upper bounds deps, not the linkage checker
|
This enforcer config is already present in the shared-config since at least version 0.3.0 as can be seen here |
Towards #6396