8000 Add "ember-cli-jshint" dependency to "app" blueprint by Turbo87 · Pull Request #5757 · ember-cli/ember-cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add "ember-cli-jshint" dependency to "app" blueprint #5757

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

Merged
merged 1 commit into from
Apr 9, 2016

Conversation

Turbo87
Copy link
Member
@Turbo87 Turbo87 commented Apr 8, 2016

This PR adds ember-cli-jshint@1.0.0 as a dependency to the "app" and "addon" blueprints. This change should makes it easier to replace JSHint with ESLint in the future.

@stefanpenner
Copy link
Contributor

I'm good with this, it seems reasonable to lift this dep to the root but would love @rwjblue input

@rwjblue
Copy link
Member
rwjblue commented Apr 9, 2016

@homu r+

@homu
Copy link
Contributor
homu commented Apr 9, 2016

📌 Commit 67bee91 has been approved by rwjblue

homu added a commit that referenced this pull request Apr 9, 2016
Add "ember-cli-jshint" dependency to "app" blueprint

This PR adds `ember-cli-jshint@1.0.0` as a dependency to the "app" and "addon" blueprints. This change should makes it easier to replace JSHint with ESLint in the future.
@homu
Copy link
Contributor
homu commented Apr 9, 2016

⌛ Testing commit 67bee91 with merge da5c4ca...

@homu
Copy link
Contributor
homu commented Apr 9, 2016

☀️ Test successful - status

@homu homu merged commit 67bee91 into ember-cli:master Apr 9, 2016
@Turbo87 Turbo87 deleted the jshint branch April 9, 2016 20:01
@Turbo87 Turbo87 modified the milestone: v2.5.0 Apr 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0