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

[Deps] Remove unecessary dependencies #2248

Merged
merged 5 commits into from
Feb 5, 2024

Conversation

Desplandis
Copy link
Contributor

@Desplandis Desplandis commented Dec 22, 2023

Description

Follow-up of #2252 (and proposal #2256) which updates and formalizes the web browser requirements of iTowns.

Motivation and Context

Following the target update in #2252, the following polyfills are not necessary anymore:

  • TextEncoder polyfill
  • async/await polyfill
  • babel's null coalescing support plugin

Moreover, the following polyfills on webpack's generated bundle are not needed anymore:

  • fetch polyfill
  • URL polyfill

This list may not be exhaustive and I may have forgotten other polyfills.

@Desplandis Desplandis force-pushed the deps/remove-old-polyfill branch from d854916 to 43d0d7a Compare January 24, 2024 21:15
@Desplandis Desplandis marked this pull request as ready for review January 24, 2024 21:20
@jailln jailln self-assigned this Feb 1, 2024
@jailln jailln self-requested a review February 1, 2024 15:59
@Desplandis Desplandis force-pushed the deps/remove-old-polyfill branch from 43d0d7a to 53dd30c Compare February 5, 2024 15:48
@Desplandis Desplandis force-pushed the deps/remove-old-polyfill branch from f9beac5 to a7c0209 Compare February 5, 2024 16:19
@Desplandis Desplandis merged commit 96b870a into iTowns:master Feb 5, 2024
9 checks passed
@Desplandis Desplandis deleted the deps/remove-old-polyfill branch February 5, 2024 16:33
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.

2 participants