Adding licence headers #55

Merged
merged 2 commits into from Feb 27, 2014

Conversation

Projects
None yet
2 participants
@sungjuly
Member

sungjuly commented Feb 17, 2014

https://issues.apache.org/jira/browse/USERGRID-4

I'm adding license headers of files (java, xml, properties, etc). Also adding apache-rat maven plugin.
Overall review, please. However, we have to avoid license conflict in Inflector class ( remove or replace something )
https://github.com/usergrid/usergrid/blob/master/stack/core/src/main/java/org/apache/usergrid/utils/Inflector.java

@zznate

This comment has been minimized.

Show comment
Hide comment
@zznate

zznate Feb 17, 2014

Added https://issues.apache.org/jira/browse/USERGRID-83 for tracking that Inflector issue. Unfortunately, that is mission critical in the polymorphic path-to-service lookup code.

I would suggest merging the above patch anyway so we don't block other work.

zznate commented Feb 17, 2014

Added https://issues.apache.org/jira/browse/USERGRID-83 for tracking that Inflector issue. Unfortunately, that is mission critical in the polymorphic path-to-service lookup code.

I would suggest merging the above patch anyway so we don't block other work.

@sungjuly

This comment has been minimized.

Show comment
Hide comment
@sungjuly

sungjuly Feb 27, 2014

Member

It's resolved. - https://issues.apache.org/jira/browse/USERGRID-83
I think it's ready to merge this PR.

Member

sungjuly commented Feb 27, 2014

It's resolved. - https://issues.apache.org/jira/browse/USERGRID-83
I think it's ready to merge this PR.

@zznate

This comment has been minimized.

Show comment
Hide comment
@zznate

zznate Feb 27, 2014

LGTM. Thanks!

zznate commented Feb 27, 2014

LGTM. Thanks!

zznate added a commit that referenced this pull request Feb 27, 2014

@zznate zznate merged commit e33c928 into master Feb 27, 2014

@zznate zznate deleted the USERGRID-4 branch Feb 27, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment