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

Allow opt-ing out of fingerprinting. #930

Merged
merged 1 commit into from
Jun 8, 2014

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Jun 6, 2014

No description provided.

@rwjblue
Copy link
Member Author

rwjblue commented Jun 6, 2014

This follows the same pattern/steps for enabling/disabling as minifyCSS and minifyJS.

@stefanpenner
Copy link
Contributor

👍 (for now)

@rjackson lets talk enhanced configuration this weekend. (anyone got time to help implementation on it?)

@rwjblue
Copy link
Member Author

rwjblue commented Jun 6, 2014

@stefanpenner - If we come up with an API, I will implement it. I want them all to be consistent, and I would prefer for ember-cli to NOT have to be configured with everything under the sun...

@stefanpenner
Copy link
Contributor

My idea is a DSL like rails. So per env config DSL and let plugins register namespaces on it or something

@twokul
Copy link
Contributor

twokul commented Jun 6, 2014

@stefanpenner I have some bandwidth this weekend, let's chat

@stefanpenner
Copy link
Contributor

first pass idea at improve configuration: https://code.stypi.com/stefanpenner/ember-cli-config-2.md

rwjblue added a commit that referenced this pull request Jun 8, 2014
@rwjblue rwjblue merged commit 52dd7f1 into ember-cli:master Jun 8, 2014
@rwjblue rwjblue deleted the allow-disabling-fingerprinting branch June 8, 2014 00:57
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

3 participants