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 Beta] Drop Object.create polyfill – just use Object.create [E… #11460

Merged

Conversation

stefanpenner
Copy link
Member

…S5-only]

(╯°□°)╯︵ ʃıɟʎʃod ǝʇɐǝɹɔ ʇɔǝɾqo

@rwjblue rwjblue changed the title [CLEANUP Beta] Drop Object.create playful – just use Object.create [E… [CLEANUP Beta] Drop Object.create polyfill – just use Object.create [E… Jun 15, 2015
@rwjblue
Copy link
Member

rwjblue commented Jun 15, 2015

Needs rebase.

@@ -100,4 +100,4 @@ if (!(Object.create && !Object.create(null).hasOwnProperty)) {
create = Object.create;
}

export default create;
export default Object.create;
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't you just delete this file?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, we likely should. We are still exporting Ember.create = Object.create for global export support, but that shouldn't require this file...

@stefanpenner stefanpenner force-pushed the remove-object-create-polyfil branch 4 times, most recently from 9183a7c to 4260df4 Compare June 15, 2015 06:15
…S5-only]

(╯°□°)╯︵ ʃıɟʎʃod ǝʇɐǝɹɔ ʇɔǝɾqo
stefanpenner added a commit that referenced this pull request Jun 15, 2015
[CLEANUP Beta] Drop Object.create polyfill – just use Object.create [E…
@stefanpenner stefanpenner merged commit 4f92d0f into emberjs:master Jun 15, 2015
@stefanpenner stefanpenner deleted the remove-object-create-polyfil branch June 15, 2015 17:07
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