Wie überzeugt man eslint, dass Einstellungen definiert sind?

[pfaffman (Jay Pfaffman)]

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

Okay. Ich verwende Einstellungen. Wie überzeuge ich eslint davon, dass sie definiert sind? Ich habe mir eine Reihe anderer Beispiele angesehen (wie https://github.com/discourse/discourse-brand-header/blob/main/javascripts/discourse/initializers/initialize-mobile-decorator.js) und sie scheinen Einstellungen nicht zu definieren.

[david (David Taylor)]

Theoretisch sollte dies von der ESLint-Konfiguration gehandhabt werden, die Sie aus unseren gemeinsamen Lint-Konfigurationen importieren

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

Aber aus irgendeinem Grund scheint das nicht zu funktionieren…

[merefield (Robert)]

Ja, stell sicher, dass du das liest @pfaffman, falls du das noch nicht getan hast

lint-configs/README.md

[cvx (Jarek Radosz)]

Hm, vielleicht gibt es eine Inkompatibilität zwischen der neuesten eslint-Konfiguration und es6-Dateien. Könnten Sie prüfen, ob die Fehler weiterhin bestehen, nachdem Sie die .es6-Suffixe entfernt haben?

[pfaffman (Jay Pfaffman)]

@merefield

Ja, lies das unbedingt @pfaffman, falls du das noch nicht getan hast

lint-configs/lint-configs/README.md at main · discourse/lint-configs · GitHub

Anstatt das zu lesen, habe ich die Sachen blind aus dem Theme-Skelett kopiert. Ich glaube, ich habe das einmal gefunden, was einige des Rätsels erklärt.

@david

Theoretisch sollte dies von der eslint-Konfiguration gehandhabt werden, die du aus unseren gemeinsamen lint-konfigurationen importierst

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

Du bist sehr freundlich, dass du vorschlägst, dass meine Theorie nicht ohne Verdienst war.

@cvx

Könntest du sehen, ob die Fehler bestehen bleiben, nachdem du die .es6-Suffixe entfernt hast?

Aha. Gibt es also einen Grund, warum ich nicht alle meine es6-Dateien überall global umbenennen sollte?