fix(build): do not lint when testing on github

This commit is contained in:
Matteo Pagliazzi
2019-10-22 20:38:01 +02:00
parent 3031caffe8
commit 9afcf8d5eb
5 changed files with 31 additions and 30 deletions

View File

@@ -22,7 +22,7 @@ jobs:
npm ci npm ci
env: env:
CI: true CI: true
- run: npm run lint - run: npm run lint-no-fix
apidoc: apidoc:
runs-on: ubuntu-latest runs-on: ubuntu-latest
strategy: strategy:

View File

@@ -73,6 +73,7 @@
}, },
"scripts": { "scripts": {
"lint": "eslint --ext .js --fix . && cd website/client && npm run lint", "lint": "eslint --ext .js --fix . && cd website/client && npm run lint",
"lint-no-fix": "eslint --ext .js . && cd website/client && npm run lint --no-fix",
"test": "npm run lint && gulp test && gulp apidoc", "test": "npm run lint && gulp test && gulp apidoc",
"test:build": "gulp test:prepare:build", "test:build": "gulp test:prepare:build",
"test:api-v3": "gulp test:api-v3", "test:api-v3": "gulp test:api-v3",

View File

@@ -214,12 +214,12 @@ const NOTIFICATIONS = {
}, },
ACHIEVEMENT_MONSTER_MAGUS: { ACHIEVEMENT_MONSTER_MAGUS: {
achievement: true, achievement: true,
label: ($t) => `${$t('achievement')}: ${$t('achievementMonsterMagus')}`, label: $t => `${$t('achievement')}: ${$t('achievementMonsterMagus')}`,
modalId: 'generic-achievement', modalId: 'generic-achievement',
}, },
ACHIEVEMENT_UNDEAD_UNDERTAKER: { ACHIEVEMENT_UNDEAD_UNDERTAKER: {
achievement: true, achievement: true,
label: ($t) => `${$t('achievement')}: ${$t('achievementUndeadUndertaker')}`, label: $t => `${$t('achievement')}: ${$t('achievementUndeadUndertaker')}`,
modalId: 'generic-achievement', modalId: 'generic-achievement',
}, },
}; };

View File

@@ -268,9 +268,9 @@ export const ANIMAL_COLOR_ACHIEVEMENTS = [
color: 'Base', petAchievement: 'backToBasics', petNotificationType: 'ACHIEVEMENT_BACK_TO_BASICS', mountAchievement: 'allYourBase', mountNotificationType: 'ACHIEVEMENT_ALL_YOUR_BASE', color: 'Base', petAchievement: 'backToBasics', petNotificationType: 'ACHIEVEMENT_BACK_TO_BASICS', mountAchievement: 'allYourBase', mountNotificationType: 'ACHIEVEMENT_ALL_YOUR_BASE',
}, },
{ {
color: 'Desert', petAchievement: 'dustDevil', petNotificationType: 'ACHIEVEMENT_DUST_DEVIL', mountAchievement: 'aridAuthority', mountNotificationType: 'ACHIEVEMENT_ARID_AUTHORITY' color: 'Desert', petAchievement: 'dustDevil', petNotificationType: 'ACHIEVEMENT_DUST_DEVIL', mountAchievement: 'aridAuthority', mountNotificationType: 'ACHIEVEMENT_ARID_AUTHORITY',
}, },
{ {
color: 'Zombie', petAchievement: 'monsterMagus', petNotificationType: 'ACHIEVEMENT_MONSTER_MAGUS', mountAchievement: 'undeadUndertaker', mountNotificationType: 'ACHIEVEMENT_UNDEAD_UNDERTAKER' color: 'Zombie', petAchievement: 'monsterMagus', petNotificationType: 'ACHIEVEMENT_MONSTER_MAGUS', mountAchievement: 'undeadUndertaker', mountNotificationType: 'ACHIEVEMENT_UNDEAD_UNDERTAKER',
}, },
]; ];

View File

@@ -205,7 +205,7 @@ function _getBasicAchievements (user, language) {
_addUltimateGear(result, user, { path: 'warrior', language }); _addUltimateGear(result, user, { path: 'warrior', language });
_addUltimateGear(result, user, { path: 'mage', altPath: 'wizard', language }); _addUltimateGear(result, user, { path: 'mage', altPath: 'wizard', language });
let cardAchievements = ['greeting', 'thankyou', 'birthday', 'congrats', 'getwell', 'goodluck']; const cardAchievements = ['greeting', 'thankyou', 'birthday', 'congrats', 'getwell', 'goodluck'];
cardAchievements.forEach(path => { cardAchievements.forEach(path => {
_addSimpleWithCount(result, user, { path, key: `${path}Cards`, language }); _addSimpleWithCount(result, user, { path, key: `${path}Cards`, language });
}); });