chrisdickinson/raf

Add license header to dist files

raphaelokon opened this issue · 4 comments

raphaelokon commented

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

ljharb commented

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

raphaelokon commented

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

If that is enough I am happy to close this

chrisdickinson commented

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

  • 👍1