Travis falla debido a eslint

Desarrollador
1

Tengo dos plugins que están fallando en las pruebas de Travis. Parece que no tengo archivos ES6 en ellos y ESLint se siente excluido.

Ejecutando linters/prettyfiers
eslint yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/eslint -v
v6.8.0
Completado en 0.32s.
prettier yarn run v1.22.5
$ /var/www/discourse/node_modules/.bin/prettier -v
2.1.1
Completado en 0.28s.
[2020-09-07 01:34:21] bundle exec rubocop --parallel plugins/discourse-allow-pm-to-staff
Inspeccionando 2 archivos
..
2 archivos inspeccionados, no se detectaron infracciones
[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
¡Ups! Algo salió mal :(
ESLint: 6.8.0.
No se encontraron archivos que coincidan con el patrón "plugins/discourse-allow-pm-to-staff".
Verifique si hay errores de escritura en el patrón.
Error: El comando falló con código de salida 2.
Visite https://yarnpkg.com/en/docs/cli/run para obtener documentación sobre este comando.
Listando infracciones de prettier en discourse-allow-pm-to-staff:
2

Sí, eslint no le gusta no tener nada que hacer. Esto lo hace sentir más útil.

mkdir -p assets/javascripts/discourse
touch assets/javascripts/discourse/eslint.js.es6
git add assets/javascripts/discourse/eslint.js.es6
git commit -m 'intentar que eslint no se sienta excluido'
git push

Ah, pero creo que esta es la verdadera solución:

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

3

Ah, pero ¿cómo vas a hacer la PR sin que eslint se queje? :wink:

4

Entiendo tan poco de lo que está pasando aquí que casi no capté tu chiste.

5

Sí, apliqué esta corrección para GitHub Actions el otro día:

Básicamente, la nueva versión de ESLint no acepta cuando no coincide con nada. Aproveché también para simplificar los comandos y aplicaré lo mismo a nuestra tarea de Rake, sin problema.

6

Eso debería solucionarlo:

Avísame cómo te va :+1:

7 
2 archivos inspeccionados, no se detectaron infracciones
[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
¡Vaya! Algo salió mal :(
ESLint: 6.8.0.
No se encontraron archivos que coincidan con el patrón "plugins/discourse-topic-default-tag".
Por favor, verifica si hay errores de escritura en el patrón.
Error: El comando falló con código de salida 2.
Para obtener información sobre este comando, visita https://yarnpkg.com/en/docs/cli/run
Listando infracciones de prettier en discourse-topic-default-tag:

Vaya. Fue mi culpa.

8

Ah, sí, claro. Está usando las funcionalidades de discourse-plugin-ci. Si revisas el código, verás:

Y mi commit aún no aparece en tests-passed.