Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change: [M3-8493] - update browserlist to latest version #10836

Merged
merged 4 commits into from
Aug 27, 2024

Conversation

abailly-akamai
Copy link
Contributor

@abailly-akamai abailly-akamai commented Aug 26, 2024

Description 📝

Small PR to update browserlist and remove the ie9 item (which has no effect on the supported browsers since deprecated) abd replaced that with a 'not-dead' param.

I consider removing it completely considering we are using vite (and this is more of a babel support tool), but perhaps we still have a couple tools relying on it 🤷

Regular updates are recommended (see: https://github.com/browserslist/browserslist#readme) and will remove some noise in our terminal.

Screenshot 2024-08-26 at 15 51 02

Here is the updated support list after this update:

and_chr 127
and_ff 127
and_qq 14.9
and_uc 15.5
android 127
chrome 127
chrome 126
chrome 125
chrome 109
edge 127
edge 126
firefox 129
firefox 128
firefox 127
firefox 115
ios_saf 17.5
ios_saf 17.4
ios_saf 16.6-16.7
ios_saf 15.6-15.8
kaios 3.0-3.1
kaios 2.5
op_mini all
op_mob 80
opera 111
opera 110
safari 17.5
safari 17.4
samsung 25
samsung 24

Changes 🔄

List any change relevant to the reviewer.

  • update browserlist by running npx update-browserslist-db@latest
  • modernize browser support list

How to test 🧪

Not sure how to verify this change since I have the suspicion it does nothing. That being said the terminal warning will be gone so there is that.

As an Author I have considered 🤔

Check all that apply

  • 👀 Doing a self review
  • ❔ Our contribution guidelines
  • 🤏 Splitting feature into small PRs
  • ➕ Adding a changeset
  • 🧪 Providing/Improving test coverage
  • 🔐 Removing all sensitive information from the code and PR description
  • 🚩 Using a feature flag to protect the release
  • 👣 Providing comprehensive reproduction steps
  • 📑 Providing or updating our documentation
  • 🕛 Scheduling a pair reviewing session
  • 📱 Providing mobile support
  • ♿ Providing accessibility support

@abailly-akamai abailly-akamai self-assigned this Aug 26, 2024
@abailly-akamai abailly-akamai marked this pull request as ready for review August 26, 2024 20:19
@abailly-akamai abailly-akamai requested a review from a team as a code owner August 26, 2024 20:19
@abailly-akamai abailly-akamai requested review from dwiley-akamai and AzureLatte and removed request for a team August 26, 2024 20:19
Copy link

Coverage Report:
Base Coverage: 82.62%
Current Coverage: 82.62%

Copy link
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed the terminal warning is now gone ✅

@abailly-akamai abailly-akamai merged commit 4d35ab7 into linode:develop Aug 27, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants