search

web-animations / web-animations-js-legacy

The original emulator of the Web Animations specification. Please use web-animations-js instead:
https://github.com/web-animations/web-animations-js
Apache License 2.0
707 stars 84 forks source link

Several scripts are missing license header #641

Closed dzhioev closed 10 years ago

dzhioev commented 10 years ago

Hello, I'm adding a copy of web-animations-js to Chromium codebase, and I have a little problem. We have a rule that every third-party file containing a source code should have a licence header. The script that enforces this rule has detected that several files in the web-animations-js repository don't have such header:

'.travis-setup.sh' 'run-lint.sh' 'run-tests.sh' '.travis-run.sh' 'tools/python/run-tests.py' 'tools/python/setup.sh' 'tools/android/config.sh' 'tools/android/stop.sh' 'tools/android/setup.sh' 'run-tests-android.sh'

Could you please add an license header to these files?

alancutter commented 10 years ago

These files are probably of little use to you for embedding web-animation-js in Chromium as they're purely Javascript testing infrastructure code. I'd just remove them as part of porting the project across.

As a side note; all work on the Web Animations polyfill has moved on to https://github.com/web-animations/web-animations-next so this repository won't see much change in the near future. You should feel free to make changes on your own fork of the project without fear of falling behind tip-of-tree.