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
Use const/let in all blueprints #6889
Use const/let in all blueprints #6889
Conversation
Changes look good. I agree with the Not sure why the tests are failing, might need to dig into the travis output to see... |
@rwjblue The two failing tests are the ones I was mentioning above, requiring the changes in ember-cli/ember-cli-blueprint-test-helpers#104 to succeed. Once there is a new release of |
@simonihmig I just published v0.17.1 of the blueprint-test-helpers |
@homu r+ |
📌 Commit 8422900 has been approved by |
Use const/let in all blueprints This will remove all left over usage of `var` off the blueprints. When to use `const` vs. `let` is a matter of opinions obviously. After checking with @Turbo87 on Slack, I was assuming the `ember-suave`-like rules of `const` in module root, `let` otherwise would be appropriate. Although in some cases we have `require` statements in a function body, so used `const` there as well. Hope that makes sense... There will be some test failures, as this is relying on ember-cli/ember-cli-blueprint-test-helpers#104 to be merged and released. So don't merge yet!
💔 Test failed - status |
@homu retry |
Use const/let in all blueprints This will remove all left over usage of `var` off the blueprints. When to use `const` vs. `let` is a matter of opinions obviously. After checking with @Turbo87 on Slack, I was assuming the `ember-suave`-like rules of `const` in module root, `let` otherwise would be appropriate. Although in some cases we have `require` statements in a function body, so used `const` there as well. Hope that makes sense... There will be some test failures, as this is relying on ember-cli/ember-cli-blueprint-test-helpers#104 to be merged and released. So don't merge yet!
☀️ Test successful - status |
This will remove all left over usage of
var
off the blueprints.When to use
const
vs.let
is a matter of opinions obviously. After checking with @Turbo87 on Slack, I was assuming theember-suave
-like rules ofconst
in module root,let
otherwise would be appropriate. Although in some cases we haverequire
statements in a function body, so usedconst
there as well. Hope that makes sense...There will be some test failures, as this is relying on ember-cli/ember-cli-blueprint-test-helpers#104 to be merged and released. So don't merge yet!