search

medialize / ally.js

JavaScript library to help modern web applications with accessibility concerns
http://allyjs.io/
MIT License
1.53k stars 83 forks source link

Build/adding rollupify #137

Closed rodneyrehm closed 7 years ago

rodneyrehm commented 8 years ago

This PR introduces rollupify to the browserify build step. This simple change achieves a reduction of file size from 127K to 76K (28K to 23K gzipped). To verify everything still works as expected, test/browser.bundle was introduced to run the unit tests on the bundle, rather than the AMD files.

The downside of this change is npm run analyze:bundle has become quite meaningless, as it now shows ally.js as one huge file, rather than many smaller packages.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 98.972% when pulling 829484ed31adac2d3ee2b9dfe9db084bc73e6cb9 on build/adding-rollupify into e46cd6d29d089bc9158147d0bd7a8aa8b8aee802 on master.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 98.972% when pulling 9687d8ca1d8c8b19fc9047e6a614c94d6793e57a on build/adding-rollupify into 56f16083842a00a1ea283ffd8e64d4510b8a46cb on master.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 98.872% when pulling ece93a6e2f06f306c15f9939a248290f9c72d93a on build/adding-rollupify into 5d111e935b4fbd11c16009eff466b0b7d5c07890 on master.