search

krambuhl / custom-event-polyfill

Polyfill for creating CustomEvents on IE9/10/11 if native implementation is missing.
MIT License
112 stars 19 forks source link

AppVeyor integration #15

Closed limonte closed 5 years ago

limonte commented 6 years ago

From https://engineering.videoblocks.com/these-four-clean-code-tips-will-dramatically-improve-your-engineering-teams-productivity-b5bd121dd150

“If it isn’t tested, it’s broken”

There are tests, but you're not running them anywhere and users can't be sure that the polyfill tests are passing. AppVeyor is the default solution for IE tests.

kumarharsh commented 5 years ago

@limonte I've added the tests in my fork - they'll be added here soon.

limonte commented 5 years ago

Great! I will reopen this issue then :)

kumarharsh commented 5 years ago

Completed the integration and published to https://npmjs.com/package/custom-event-polyfill