Hi,

+1 binding. You may want consider some changes to LICENSE / NOTICE for the next 
release.

I checked:
- file contain incubating
- DISCLAIMER exists
- LICENSE and NOTICE have minor issues (see below)
- all source files have headers
- no unexpected binary files
- can compile source release

For LICENSE:
- Missing MIT licence normalize.css
- Not obvious that Units, CMP or go-codec is contained in the source release. 
If so they are not required to be mentioned in LICENSE.

It's best to try a keep content of NOTICE to a minimum. [1]
- No need for NOTICE to contain reference to license (everything is permissive)
- The Owl logo information should be in LICENSE I think

Thanks,
Justin

1.http://www.apache.org/dev/licensing-howto.html#mod-notice
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to