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

[BREAKING ENHANCEMENT] Update ember-cli-content-security-policy to v0.4.0 #3571

Merged
merged 1 commit into from
Mar 20, 2015

Conversation

sir-dunxalot
Copy link
Contributor

This PR updates the version of ember-cli-content-security-policy that vanilla Ember apps are generated with.

The benefit of this is to avoid rwjblue/ember-cli-content-security-policy#27, which was creating unnecessary console errors for Ember users.

@sir-dunxalot sir-dunxalot changed the title [BREAKING ENHANCEMENT] Update ember-content-security-policy to v0.4.0 [BREAKING ENHANCEMENT] Update ember-cli-content-security-policy to v0.4.0 Mar 19, 2015
@sir-dunxalot
Copy link
Contributor Author

Not sure why the Travis build failed - the failing test seems somewhat unrelated to my one line change. Everything is passing on my local so is there something else I should be aware of?

@rwjblue
Copy link
Member

rwjblue commented Mar 20, 2015

How is this a breaking change? The diff indicates that the changes were only additive....

@sir-dunxalot
Copy link
Contributor Author

From CONTRIBUTING.md:

If a change requires a user to change their configuration, bower.json, package.json or Brocfile.js also add a BREAKING tag within the brackets before any other tags (example [BREAKING BUGFIX]).

Did I misread that?

@rwjblue
Copy link
Member

rwjblue commented Mar 20, 2015

HA! Nope, you got it right. I think everyone else needs to read the guide though.. 😉

Thank you!

rwjblue added a commit that referenced this pull request Mar 20, 2015
[BREAKING ENHANCEMENT] Update ember-cli-content-security-policy to v0.4.0
@rwjblue rwjblue merged commit 237cc60 into ember-cli:master Mar 20, 2015
@sir-dunxalot
Copy link
Contributor Author

Honestly I nearly missed it too haha. Thanks for the quick turn around (and all your work with Ember CLI!).

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

2 participants