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

[CLEANUP] Remove deprecated assign polyfill #20380

Merged
merged 1 commit into from Mar 21, 2023
Merged

Conversation

chriskrycho
Copy link
Contributor

  • Remove the polyfill itself.
  • Remove the entire @ember/polyfills package, which is now empty.
  • Remove the Ember.platform API, which has loooong been unused.

@chriskrycho chriskrycho force-pushed the remove-ember-assign branch 2 times, most recently from 48131a9 to 0e4c831 Compare February 21, 2023 15:48
@wagenet
Copy link
Member

wagenet commented Feb 21, 2023

Looks like you missed a reexports test.

@chriskrycho
Copy link
Contributor Author

Thanks – will fix that up shortly!

- Remove the polyfill itself.
- Remove the entire `@ember/polyfills` package, which is now empty.
- Remove the `Ember.platform` API, which has loooong been unused.
@wagenet wagenet merged commit fd0ea36 into master Mar 21, 2023
15 of 16 checks passed
@wagenet wagenet deleted the remove-ember-assign branch March 21, 2023 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants