-
Notifications
You must be signed in to change notification settings - Fork 30
build: use shared browserslist configuration #220
build: use shared browserslist configuration #220
Conversation
Thanks for the pull request, @Mashal-m! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
Codecov ReportBase: 50.00% // Head: 50.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #220 +/- ##
=======================================
Coverage 50.00% 50.00%
=======================================
Files 15 15
Lines 164 164
Branches 28 28
=======================================
Hits 82 82
Misses 77 77
Partials 5 5 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@Mashal-m - when you are ready to have this reviewed, please tag Colin Brash. Thank you! |
Ready to review. |
Hi @colinbrash - friendly ping on this :) Could you please take a look? |
…merce into mashal-m/add-browserslist-config
Ticket:
Use shared @edx/browserslist-config
browserslist
configuration in favor of using a shared configuration.is-es5
check in CI since ES5 was only needed for IE 11 support which is dropped. The supported browsers defined by the shared configuration all natively support ES6.This change reduces the resultant asset bundle size.