eslint로 인해 Travis 테스트 실패

개발
1

트래비스 테스트가 실패하는 플러그인이 두 개 있습니다. 해당 플러그인에는 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 'try to make eslint not feel left out'
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 상태가 아니에요.