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

[2.X] Deprecate vendor-prefix and vendor-suffix. #5309

Merged
merged 2 commits into from Jan 26, 2016

Conversation

nathanhammond
Copy link
Contributor

These hooks make it easy to do bad things inside of ember-cli. Deprecating them to make the world a better place.

Don't merge yet, need to come up with a solution for https://github.com/rwjblue/ember-disable-prototype-extensions first.

@nathanhammond nathanhammond changed the title Deprecate vendor-prefix and vendor-suffix. [2.X] Deprecate vendor-prefix and vendor-suffix. Jan 6, 2016
@stefanpenner
Copy link
Contributor

👍

@rwjblue
Copy link
Member

rwjblue commented Jan 6, 2016

I'm happy to backport the deprecation to 1-13 branch also.

@nathanhammond
Copy link
Contributor Author

Landable if we bump the version of https://github.com/rwjblue/ember-disable-prototype-extensions. :)

@rwjblue
Copy link
Member

rwjblue commented Jan 7, 2016

% npm publish
+ ember-disable-prototype-extensions@1.1.0

@nathanhammond
Copy link
Contributor Author

:shipit: if you wish.

@nathanhammond
Copy link
Contributor Author

Can we restart travis-ci and appveyor? This should pass.

@stefanpenner
Copy link
Contributor

gave them a kick

@stefanpenner
Copy link
Contributor

@homu r+

@homu
Copy link
Contributor

homu commented Jan 26, 2016

📌 Commit c481080 has been approved by stefanpenner

@homu
Copy link
Contributor

homu commented Jan 26, 2016

⌛ Testing commit c481080 with merge 0b87733...

homu added a commit that referenced this pull request Jan 26, 2016
…enner

[2.X] Deprecate vendor-prefix and vendor-suffix.

These hooks make it easy to do bad things inside of ember-cli. Deprecating them to make the world a better place.

Don't merge yet, need to come up with a solution for https://github.com/rwjblue/ember-disable-prototype-extensions first.
@homu
Copy link
Contributor

homu commented Jan 26, 2016

☀️ Test successful - status

@homu homu merged commit c481080 into ember-cli:master Jan 26, 2016
@homu homu mentioned this pull request Jan 26, 2016
@nathanhammond nathanhammond deleted the deprecate-vendor-hooks branch October 16, 2016 17:13
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.

None yet

4 participants