search

chrisdickinson / raf

requestAnimationFrame polyfill library
MIT License
744 stars 54 forks source link

Add license header to dist files #32

Closed raphaelokon closed 5 years ago

raphaelokon commented 8 years ago

Add license header to dest files so that imports implicitly import the license portion.

ljharb commented 8 years ago

Why is a license needed in the file itself? There's one in the repo already. That seems redundant.

raphaelokon commented 8 years ago

When you import the module and bundle/uglify the code (here with Rollup) there is literally no reference in the code describing/linking to the repo/license file itself. All my other deps (Flickity, jQuery, loads et al.) just do it and their respective licenses are preserved. I added a license field manually in my code where I use …

   /*! raf https://github.com/chrisdickinson/raf @license: MIT license*/
   raf.polyfill();
raphaelokon commented 8 years ago

If that is enough I am happy to close this

chrisdickinson commented 5 years ago

(I'm going to close this since a LICENSE file has been added.)