Skip to content

fix: remove polyfills which are out of range of our engines#35

Merged
lukekarrys merged 1 commit intomainfrom
lk/rm-old-polyfills
Jul 20, 2022
Merged

fix: remove polyfills which are out of range of our engines#35
lukekarrys merged 1 commit intomainfrom
lk/rm-old-polyfills

Conversation

@lukekarrys
Copy link
Copy Markdown
Contributor

The polyfills removed are:

  • file-url-to-path.js
  • mkdir.js

@lukekarrys lukekarrys requested a review from a team as a code owner July 20, 2022 20:45
@wraithgar
Copy link
Copy Markdown
Contributor

How is this a feat? What's the new feature?

@lukekarrys
Copy link
Copy Markdown
Contributor Author

Yeah, it felt like a blurry line here to me. The feature is that it's not going to use polyfills anymore for those methods 😄 (which would only happen if you're running it outside of engines support). That didn't feel like a fix to me.

@lukekarrys lukekarrys force-pushed the lk/rm-old-polyfills branch from d7aeaaf to 922676f Compare July 20, 2022 21:48
@lukekarrys lukekarrys changed the title feat: remove polyfills which are out of range of our engines fix: remove polyfills which are out of range of our engines Jul 20, 2022
@lukekarrys
Copy link
Copy Markdown
Contributor Author

Switched commit and PR title to fix.

@lukekarrys lukekarrys force-pushed the lk/rm-old-polyfills branch 2 times, most recently from 71e562d to 6fb1fe8 Compare July 20, 2022 21:56
The polyfills removed are:
- file-url-to-path.js
- mkdir.js
@lukekarrys lukekarrys force-pushed the lk/rm-old-polyfills branch from 6fb1fe8 to 5950a8a Compare July 20, 2022 22:11
@lukekarrys lukekarrys merged commit be1e7b2 into main Jul 20, 2022
@lukekarrys lukekarrys deleted the lk/rm-old-polyfills branch July 20, 2022 22: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