Closed ijlee2 closed 4 years ago
General notes: Hi, everyone. To allow more people to participate in Hacktoberfest, please claim only 1 issue labeled with Hacktoberfest
in this repo (unless there are no others who can help). 🙂
Please check #hacktoberfest
on Discord to find other repos that can use your help. Thanks and happy Hacktoberfest!
@ijlee2 I am Ember JS Developer and also wanted to contribute. Can you please guide me.
@takshch I updated the TODO instructions. Will you have a look?
Problem
Once #197 has been completed and merged, we can leveragenpm-run-all
to parallelize lints and tests.After talking to @takshch, I think we can solve this issue in a slightly different manner. I've updated the TODOs below.
TODOs
Part 2. Update CI
[ ] Install
npm-run-all
.[ ] Uninstall
ember-cli-template-lint
andember-cli-eslint
. Afterwards, installember-template-lint
andeslint
.If you run
npm run lint:hbs
ornpm run lint:js
, all lint checks should continue to pass.[ ] In
package.json
, create alint
script and update thetest
script to run tasks in parallel:[ ] Simplify
.travis.yml
(CI script) to runnpm run lint
. Push a commit to check that the CI continues to pass.