Fixed Travis to exit early on lint failures and skip duplicate linting in ember tests

no issue
- Travis runs all `script` tasks even if an earlier one fails so moved to using `&&` so ember tests aren't run if linting errors
- Added `ember-cli-eslint` to the addons blacklist when building in Travis so that we aren't running linting twice (also reduces build log output)
This commit is contained in:
Kevin Ansfield 2019-03-11 15:45:26 +00:00
parent a0e9422699
commit 02f73bc44e
2 changed files with 7 additions and 2 deletions

View File

@ -31,8 +31,7 @@ install:
- yarn
script:
- npm run lint:js
- COVERAGE=true npm test
- npm run lint:js && COVERAGE=true npm test
after_success:
- npm run coverage

View File

@ -105,8 +105,14 @@ const simplemdeAssets = function () {
return config;
};
let blacklist = [];
if (process.env.CI) {
blacklist.push('ember-cli-eslint');
}
module.exports = function (defaults) {
let app = new EmberApp(defaults, {
addons: {blacklist},
'ember-cli-babel': {
optional: ['es6.spec.symbols'],
includePolyfill: false