Travis falhando devido ao eslint

Dev
1

Tenho dois plugins que estão falhando nos testes do Travis. Parece que não há nenhum arquivo ES6 neles e o eslint se sente excluído?

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:
1 curtida
2

Sim, o eslint não gosta de ficar sem nada para fazer. Isso o faz parecer mais útil.

mkdir -p assets/javascripts/discourse
touch assets/javascripts/discourse/eslint.js.es6
git add assets/javascripts/discourse/eslint.js.es6
git commit -m 'tente fazer o eslint não se sentir excluído'
git push

Ah, mas acho que esta é a solução real:

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

1 curtida
3

Ah, mas como você vai fazer o PR disso sem o eslint reclamar? :wink:

1 curtida
4

Entendo tão pouco do que está acontecendo aqui que quase não entendi sua piada!

1 curtida
5

Sim, apliquei essa correção para o GitHub Actions no outro dia:

Basicamente, o novo eslint não gosta quando não há correspondência. Também aproveitei para simplificar os comandos e aplicarei o mesmo à nossa tarefa rake, sem problemas.

7 curtidas
6

Isso deve resolver:

Me avise como ficou :+1:

4 curtidas
7 
2 arquivos inspecionados, nenhuma ofensa detectada
[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
Ops! Algo deu errado! :(
ESLint: 6.8.0.
Nenhum arquivo correspondente ao padrão "plugins/discourse-topic-default-tag" foi encontrado.
Verifique se há erros de digitação no padrão.
Erro: comando falhou com código de saída 2.
Visite https://yarnpkg.com/en/docs/cli/run para documentação sobre este comando.
Listando ofensas do prettier em discourse-topic-default-tag:

Ops. Minha culpa.

8

Ah, certo, sim, está usando o discourse-plugin-ci. Se você ler o código, verá:

E meu commit ainda não está em tests-passed.

3 curtidas
9

Este tópico foi automaticamente fechado 2 dias após a última resposta. Novas respostas não são mais permitidas.