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
ZOOKEEPER-3440 Fix Apache RAT check by excluding binary files (images) #998
Conversation
eolivelli
commented
Jun 22, 2019
•
edited
edited
- Fix Apache Rat plugin configuration (make it pass against C client, docs and contrib module)
- Add apache-rat:check on Travis
@anmolnar @nkalmar @phunt the patch is now ready, as already said on JIRA (https://issues.apache.org/jira/browse/ZOOKEEPER-3440) I don't like that exclusions we should take care of cleaning up the contrib module By now we can fix the pom as I am suggesting |
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.
LGTM!
(And I'm all +1 on moving contrib out of ZK repo)
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.
+1
Should this fix go to 3.5 as well? Looks to me it should have the same problem. |
@nkalmar I image so, as we are using Maven in zk 3.5 it is better to apply the fix there. |
- Fix Apache Rat plugin configuration (make it pass against C client, docs and contrib module) - Add apache-rat:check on Travis Author: Enrico Olivelli <eolivelli@apache.org> Reviewers: Andor Molnár <andor@apache.org>, Norbert Kalmar <nkalmar@yahoo.com>, Michael Han <hanm@apache.org> Closes apache#998 from eolivelli/fix/ZOOKEEPER-3440-rat
- Fix Apache Rat plugin configuration (make it pass against C client, docs and contrib module) - Add apache-rat:check on Travis Author: Enrico Olivelli <eolivelli@apache.org> Reviewers: Andor Molnár <andor@apache.org>, Norbert Kalmar <nkalmar@yahoo.com>, Michael Han <hanm@apache.org> Closes apache#998 from eolivelli/fix/ZOOKEEPER-3440-rat
- Fix Apache Rat plugin configuration (make it pass against C client, docs and contrib module) - Add apache-rat:check on Travis Author: Enrico Olivelli <eolivelli@apache.org> Reviewers: Andor Molnár <andor@apache.org>, Norbert Kalmar <nkalmar@yahoo.com>, Michael Han <hanm@apache.org> Closes apache#998 from eolivelli/fix/ZOOKEEPER-3440-rat