Travis のテストが eslint により失敗しています

開発
1

2 つのプラグインで Travis のテストが失敗しています。それらに ES6 ファイルが含まれていないため、eslint が寂しがっているようです。

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

はい、eslint は何もするものがなくなると嫌がります。これでより有用に感じられます。

mkdir -p assets/javascripts/discourse
touch assets/javascripts/discourse/eslint.js.es6
git add assets/javascripts/discourse/eslint.js.es6
git commit -m 'eslint が取り残されたと感じないようにする'
git push

ああ、でもこれが本当の解決策だと思います：

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

3

ああ、でも eslint が文句を言わずに PR するのはどうするの？ :wink:

4

ここで何が起きているのかほとんど理解できていなかったので、あなたの冗談に気づくところでした！

5

はい、先日 GitHub Actions 向けのこの修正をプッシュしました：

つまり、新しい eslint は一致するものがない場合にエラーを出すようになりました。この機会にコマンドも簡素化しましたので、rake タスクにも同じように適用します。ご安心ください。

6

これで解決するはずです:

結果を教えてください :+1:

7 
2 files inspected, no offenses detected
[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
Oops! Something went wrong! :(
ESLint: 6.8.0.
No files matching the pattern "plugins/discourse-topic-default-tag" 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-topic-default-tag:

おっと、私のミスです。

8

ああ、そうそう。discourse-plugin-ci の仕組みを使っているんだ。コードを見てみるとわかるよ：

そして、私のコミットはまだ tests-passed に反映されていないんだ。