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

[1.X] Deprecate vendor-prefix and vendor-suffix. #5310

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.

@stefanpenner
Copy link
Contributor

👍

@rwjblue
Copy link
Member

rwjblue commented Jan 6, 2016

LOL. I guess I have to review all PR's before asking for more back porting ;)

@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

@nathanhammond gave them a kick :)

@stefanpenner
Copy link
Contributor

@homu r+

@homu
Copy link
Contributor

homu commented Jan 26, 2016

📌 Commit fc25058 has been approved by stefanpenner

@homu
Copy link
Contributor

homu commented Jan 26, 2016

⌛ Testing commit fc25058 with merge 102c7b4...

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

[1.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 fc25058 into ember-cli:stable-1-13 Jan 26, 2016
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