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.

Sign up for GitHub

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

Jump to bottom

Remove old polyfills #548

Merged
merged 1 commit into from Feb 23, 2021
Merged

Remove old polyfills #548

merged 1 commit into from Feb 23, 2021

Conversation

wooorm
Copy link
Member

@wooorm wooorm commented Feb 21, 2021

  • Add restriction to ES 2015
  • Use Object.assign instead of xtend
  • Require React.Fragment
  • Remove hacks needed for React 15

@wooorm
Remove old polyfills
70ba0d5 
* Add restriction to ES 2015
* Use `Object.assign` instead of `xtend`
* Require `React.Fragment`
* Remove hacks needed for React 15
@wooorm wooorm added 🏡 area/internal This affects the hidden internals 🧑 semver/major This is a change 🦋 type/enhancement This is great to have labels Feb 21, 2021
ChristianMurphy
ChristianMurphy approved these changes Feb 22, 2021
View reviewed changes
src/ast-to-react.js Show resolved Hide resolved
@wooorm wooorm merged commit 236182e into main Feb 23, 2021
@wooorm wooorm deleted the fragment branch February 23, 2021 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChristianMurphy ChristianMurphy ChristianMurphy approved these changes

Assignees
No one assigned
Labels
🏡 area/internal This affects the hidden internals 🧑 semver/major This is a change 🦋 type/enhancement This is great to have
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wooorm @ChristianMurphy