Тесты Travis падают из-за eslint

Разработка
1

У меня два плагина, которые не проходят тесты в Travis. Похоже, в них нет файлов ES6, и eslint чувствует себя обделённым?

Запуск линтеров/форматтеров
eslint yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/eslint -v
v6.8.0
Готово за 0.32с.
prettier yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/prettier -v
2.1.1
Готово за 0.28с.
[2020-09-07 01:34:21] bundle exec rubocop --parallel plugins/discourse-allow-pm-to-staff
Проверка 2 файлов
..
Проверено 2 файла, нарушений не обнаружено
[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
Ой! Что-то пошло не так! :(
ESLint: 6.8.0.
Файлы, соответствующие шаблону "plugins/discourse-allow-pm-to-staff", не найдены.
Пожалуйста, проверьте наличие опечаток в шаблоне.
Ошибка: команда завершилась с кодом 2.
Информация: https://yarnpkg.com/en/docs/cli/run для документации по этой команде.
Список нарушений prettier в 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

Ах, но как вы планируете это PR-ить, чтобы eslint не ругался? :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

А, точно, да, там используется stuff из discourse-plugin-ci. Если вы посмотрите в код, то увидите:

А мой коммит ещё не в tests-passed.