Travis-Tests schlagen wegen eslint fehl

Entwickler
1

Ich habe zwei Plugins, bei denen die Travis-Tests fehlschlagen. Es scheint, als gäbe es keine ES6-Dateien in ihnen, und ESLint fühlt sich etwas vernachlässigt?

Running linters/prettyfiers
eslint yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/eslint -v
v6.8.0
Done in 0.32s.
prettier yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/prettier -v
2.1.1
Done in 0.28s.
[2020-09-07 01:34:21] bundle exec rubocop --parallel plugins/discourse-allow-pm-to-staff
Inspecting 2 files
..
2 files inspected, no offenses detected
[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
Oops! Something went wrong! :(
ESLint: 6.8.0.
No files matching the pattern "plugins/discourse-allow-pm-to-staff" were found.
Please check for typing mistakes in the pattern.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Listing prettier offenses in discourse-allow-pm-to-staff:
2

Ja, eslint mag es nicht, nichts zu tun zu haben. Das lässt es nützlicher wirken.

mkdir -p assets/javascripts/discourse
touch assets/javascripts/discourse/eslint.js.es6
git add assets/javascripts/discourse/eslint.js.es6
git commit -m 'Versuch, eslint nicht ausgeschlossen zu fühlen'
git push

Ach, aber ich denke, das ist die eigentliche Lösung:

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

3

Ah, aber wie willst du das PRen, ohne dass ESLint meckert? :wink:

4

Ich verstehe hier so wenig, was los ist, dass ich deinen Witz fast nicht begriffen hätte!

5

Ja, ich habe diese Korrektur für GitHub Actions vor ein paar Tagen durchgeführt:

Im Grunde mag die neue ESLint-Version es nicht, wenn nichts übereinstimmt. Ich habe auch die Gelegenheit genutzt, die Befehle zu vereinfachen und werde das Gleiche für unsere Rake-Aufgabe anwenden, keine Sorge.

6

Das sollte es beheben:

Lass mich wissen, wie es läuft :+1:

7 
2 Dateien geprüft, keine Verstöße festgestellt
[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
Hoppla! Etwas ist schiefgelaufen! :(
ESLint: 6.8.0.
Keine Dateien, die dem Muster "plugins/discourse-topic-default-tag" entsprechen, wurden gefunden.
Bitte prüfen Sie das Muster auf Tippfehler.
Fehler: Der Befehl ist mit dem Exit-Code 2 fehlgeschlagen.
info Weitere Informationen unter https://yarnpkg.com/en/docs/cli/run.
Liste der Prettier-Verstöße in discourse-topic-default-tag:

Hoppla. Das war mein Fehler.

8

Ah ja, stimmt, es nutzt die discourse-plugin-ci-Funktionen. Wenn du dir den Code ansiehst, wirst du Folgendes sehen:

Und mein Commit ist noch nicht in tests-passed.