Github user joshelser commented on the pull request:
https://github.com/apache/incubator-rya/pull/2#issuecomment-161380000
Took a glance, and it looks alright to me. I haven't put a close eye to
whether the LICENSE and NOTICE files are actually correct (not a super high
priority at Rya's current stage in life).
Lots of nice fixes to the poms. Thanks for taking the time to work on this.
A minor nit is that it would be better to push down the rat-plugin
exclusions to each module (rather than defining it in the parent pom). This
helps make sure we know _exactly_ what is being ignored and we don't
accidentally ignore something new in the future.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
