Travis échoue à cause d'eslint

Dev
1

J’ai deux plugins qui échouent aux tests Travis. Il semble que je n’ai aucun fichier ES6 et ESLint se sent exclu ?

Exécution des linters/prettyfiers
eslint yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/eslint -v
v6.8.0
Terminé en 0,32 s.
prettier yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/prettier -v
2.1.1
Terminé en 0,28 s.
[2020-09-07 01:34:21] bundle exec rubocop --parallel plugins/discourse-allow-pm-to-staff
Inspection de 2 fichiers
..
2 fichiers inspectés, aucune infraction détectée
[2020-09-07 01:34:22] bundle exec ruby script/i18n_lint.rb plugins/discourse-allow-pm-to-staff/config/locales/{client,server}.en.yml
[2020-09-07 01:34:23] yarn eslint --global I18n --ext .es6 plugins/discourse-allow-pm-to-staff
yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/eslint --global I18n --ext .es6 plugins/discourse-allow-pm-to-staff
Oups ! Quelque chose s'est mal passé :(
ESLint : 6.8.0.
Aucun fichier correspondant au motif "plugins/discourse-allow-pm-to-staff" n'a été trouvé.
Veuillez vérifier les erreurs de frappe dans le motif.
Erreur : la commande a échoué avec le code de sortie 2.
Pour plus d'informations, visitez https://yarnpkg.com/en/docs/cli/run.
Liste des infractions prettier dans discourse-allow-pm-to-staff :
2

Oui, eslint n’aime pas ne rien avoir à faire. Cela le rend plus utile.

mkdir -p assets/javascripts/discourse
touch assets/javascripts/discourse/eslint.js.es6
git add assets/javascripts/discourse/eslint.js.es6
git commit -m 'essayer de ne pas laisser eslint de côté'
git push

Ah, mais je pense que c’est la vraie solution :

https://github.com/discourse/discourse/compare/master...pfaffman:patch-5

3

Ah, mais comment allez-vous faire une PR sans que ESLint ne se plainte ? :wink:

4

Je comprends si peu ce qui se passe ici que j’ai failli ne pas comprendre ta blague !

5

Oui, j’ai appliqué cette correction pour GitHub Actions l’autre jour :

En gros, la nouvelle version d’ESLint n’apprécie pas quand aucune correspondance n’est trouvée. J’en ai aussi profité pour simplifier les commandes ; je ferai de même pour notre tâche Rake, pas de souci.

6

Cela devrait régler le problème :

Tiens-moi au courant de la suite :+1:

7 
2 fichiers inspectés, aucune infraction détectée
[2020-09-10 18:43:13] bundle exec ruby script/i18n_lint.rb plugins/discourse-topic-default-tag/config/locales/{client,server}.en.yml
[2020-09-10 18:43:13] yarn eslint --global I18n --ext .es6 plugins/discourse-topic-default-tag
yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/eslint --global I18n --ext .es6 plugins/discourse-topic-default-tag
Oups ! Quelque chose s'est mal passé :(
ESLint : 6.8.0.
Aucun fichier correspondant au motif "plugins/discourse-topic-default-tag" n'a été trouvé.
Veuillez vérifier les erreurs de frappe dans le motif.
Erreur : la commande a échoué avec le code de sortie 2.
info Visitez https://yarnpkg.com/en/docs/cli/run pour la documentation sur cette commande.
Liste des infractions prettier dans discourse-topic-default-tag :

Oups. Ma faute.

8

Ah oui, c’est vrai, ça utilise les fonctionnalités de discourse-plugin-ci. Si vous lisez le code, vous verrez :

Et mon commit n’est pas encore dans tests-passed