Hi,

+1 binding. Nice work on the LICENSE/NOTICE files. I checked that the “GPL" 
dependancy issue isn’t one as it dual licensed under GPL/CDDL.

I checked:
- name contains incubating
- signatures and hashed good
- DISCLAIMER exists
- LICENSE and NOTICE good
- All source files have ASF headers
- No unexpected binary files in source release
- can compile from source

You have a very minor issue with LICENSE in that polyfill references the glyph 
icons license and the NOTICE file has a bit of extra text in it that’s not 
required.

Thanks,
Justin
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to