Comment convaincre eslint que les paramètres sont définis ?

Dev
1

https://github.com/pfaffman/discourse-custom-homepage-for-groups/actions/runs/7211339319/job/19646550333

/home/runner/work/discourse-custom-homepage-for-groups/discourse-custom-homepage-for-groups/javascripts/discourse/initializers/custom-homepage-set.js.es6
Error:   13:11  error  'settings' is not defined  no-undef
Error:   14:21  error  'settings' is not defined  no-undef
Error:   19:20  error  'settings' is not defined  no-undef
Error:   20:21  error  'settings' is not defined  no-undef
Error:   25:36  error  'settings' is not defined  no-undef
Error:   26:21  error  'settings' is not defined  no-undef
Error:   32:40  error  'settings' is not defined  no-undef
Error:   33:26  error  'settings' is not defined  no-undef

https://github.com/pfaffman/discourse-custom-homepage-for-groups/blob/main/javascripts/discourse/initializers/custom-homepage-set.js.es6

D’accord. J’utilise settings. Comment puis-je convaincre eslint qu’ils sont définis ? J’ai examiné un certain nombre d’autres exemples (comme https://github.com/discourse/discourse-brand-header/blob/main/javascripts/discourse/initializers/initialize-mobile-decorator.js) et ils ne semblent pas définir settings.

3

En théorie, cela devrait être géré par la configuration eslint que vous importez de nos lint-configs partagés :thinking:

https://github.com/discourse/lint-configs/blob/main/lint-configs/eslint-theme.cjs#L7-L10

Mais il semble que cela ne fonctionne pas pour une raison quelconque…

2 « J'aime »
4

Oui, assurez-vous de lire ceci @pfaffman si ce n’est pas déjà fait :slight_smile:

5

Hm, peut-être qu’il y a une incompatibilité entre la dernière configuration eslint et les fichiers es6. Pourriez-vous voir si les erreurs persistent après avoir supprimé les suffixes .es6 ?

4 « J'aime »
6

Plutôt que de lire cela, j’ai copié les éléments aveuglément à partir du squelette du thème. :person_shrugging: Je pense que je l’ai trouvé une fois, ce qui explique une partie du mystère.

Vous êtes très gentil de suggérer que ma théorie n’était pas sans mérite.

image
image375×173 7.83 KB

Aha. Y a-t-il donc une raison de ne pas renommer globalement tous mes fichiers es6 partout ?

1 « J'aime »
7

Ce sujet a été automatiquement fermé 30 jours après la dernière réponse. Les nouvelles réponses ne sont plus autorisées.