videojs / videojs-contrib-ads

A Tool for Building Video.js Ad Plugins
http://videojs.github.io/videojs-contrib-ads/
Other
384 stars 257 forks source link

chore: lint to code to vjs 7 standards #433

Closed brandonocasey closed 6 years ago

brandonocasey commented 6 years ago

This removes the eslint configuration in favor of using videojs-standard which all of our other projects use. This pr is waiting for: https://github.com/videojs/standard/pull/36 to be merged so that we can turn the jsdoc rules off, even though they are just warnings and won't cause the build to fail.

misteroneill commented 6 years ago

It looks like this needs an update to .travis.yml to no longer run tests in Node 4 because videojs-standard does not work there.