Mashal-m commented 2 years ago

Ticket: Use shared @edx/browserslist-config

Removes custom browserslist configuration in favor of using a shared configuration.
Removes 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.

codecov[bot] commented 2 years ago

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (3a60955) compared to base (03fa143). Patch has no changes to coverable lines.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #273 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 108 108 Lines 1260 1260 Branches 248 248 ========================================= Hits 1260 1260 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Mashal-m commented 1 year ago

Closing this because these changes are merged now PR

openedx / frontend-app-gradebook

refactor: use shared browserslist config #273

Codecov Report