New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BEAM-3690] ban mockito-all and hamcrest-all #4712
Conversation
@lukecwik , The purpose of this change is to configure the maven enforcer plugin to ban hamcrest-all/mockito-all using https://maven.apache.org/enforcer/enforcer-rules/bannedDependencies.html. |
sdks/java/nexmark/pom.xml
Outdated
</goals> | ||
<configuration> | ||
<ignoredUnusedDeclaredDependencies> | ||
<ignoredUnusedDeclaredDependency>org.hamcrest:hamcrest-library:jar:${hamcrest.version}</ignoredUnusedDeclaredDependency> |
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 seems to duplicate the lines below.
Are they needed?
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.
Can you fix the indentation?
sdks/java/nexmark/pom.xml
Outdated
</goals> | ||
<configuration> | ||
<ignoredUnusedDeclaredDependencies> | ||
<ignoredUnusedDeclaredDependency>org.hamcrest:hamcrest-library:jar:${hamcrest.version}</ignoredUnusedDeclaredDependency> |
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.
Can you fix the indentation?
Note that we don't use tabs for indentation. I am cleaning up/flattening this PR and merging. |
See: https://issues.apache.org/jira/browse/BEAM-3690
ban mockito-all and hamcrest-all