Skip to content
This repository has been archived by the owner on May 26, 2019. It is now read-only.

Update contributions section for CLI #69

Merged
merged 2 commits into from
Mar 20, 2015

Conversation

stevekinney
Copy link
Contributor

Updated the contribution guides section on feature flags for Ember CLI.

@stevekinney stevekinney mentioned this pull request Mar 19, 2015
19 tasks
var ENV = {
EmberENV: {
FEATURES: {
Ember.FEATURES["htmlbars"] = true
Copy link
Contributor

Choose a reason for hiding this comment

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

@stevekinney
Copy link
Contributor Author

How so?

@locks
Copy link
Contributor

locks commented Mar 20, 2015

Because it's using the global Ember.FEATURES hash

@stevekinney
Copy link
Contributor Author

Yea. Duh. Good point. You are totally right.

On Thu, Mar 19, 2015 at 9:21 PM, Ricardo Mendes notifications@github.com
wrote:

Because it's using the global Ember.FEATURES hash

Reply to this email directly or view it on GitHub:
#69 (comment)

@@ -44,7 +44,7 @@ problems with early return.
```js
if (Ember.FEATURES.isEnabled("feature")) {
Copy link
Contributor

Choose a reason for hiding this comment

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

what about this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That would be in the Ember source code—not in a CLI project, I believe.

Copy link
Sponsor Member

Choose a reason for hiding this comment

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

technically it is ;-) but seems good

@locks
Copy link
Contributor

locks commented Mar 20, 2015

:shipit:

mixonic added a commit that referenced this pull request Mar 20, 2015
@mixonic mixonic merged commit b6227f8 into emberjs:master Mar 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants