pfaffman
14.Декабрь.2023 16:00:46
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
Ошибка: 13:11 ошибка 'settings' не определён no-undef
Ошибка: 14:21 ошибка 'settings' не определён no-undef
Ошибка: 19:20 ошибка 'settings' не определён no-undef
Ошибка: 20:21 ошибка 'settings' не определён no-undef
Ошибка: 25:36 ошибка 'settings' не определён no-undef
Ошибка: 26:21 ошибка 'settings' не определён no-undef
Ошибка: 32:40 ошибка 'settings' не определён no-undef
Ошибка: 33:26 ошибка 'settings' не определён no-undef
https://github.com/pfaffman/discourse-custom-homepage-for-groups/blob/main/javascripts/discourse/initializers/custom-homepage-set.js.es6
Верно. Я использую settings. Как мне убедить eslint, что они определены? Я просмотрел множество других примеров (например, https://github.com/discourse/discourse-brand-header/blob/main/javascripts/discourse/initializers/initialize-mobile-decorator.js ), и в них, похоже, settings не объявляются.
david
14.Декабрь.2023 16:07:58
3
Теоретически это должно обрабатываться конфигурацией eslint, которую вы импортируете из наших общих lint-configs
https://github.com/discourse/lint-configs/blob/main/lint-configs/eslint-theme.cjs#L7-L10
Но, похоже, по какой-то причине это не работает…
merefield
14.Декабрь.2023 16:08:42
4
Да, обязательно прочитайте это, @pfaffman , если вы еще этого не сделали
# @discourse/lint-configs
Shareable lint configs for Discourse core, plugins, and themes
## Usage
Add `@discourse/lint-configs` to package.json, and create these three files:
### eslint.config.mjs
```js
import DiscourseRecommended from "@discourse/lint-configs/eslint";
export default [...DiscourseRecommended];
```
or in themes/theme components:
```js
import DiscourseThemeRecommended from "@discourse/lint-configs/eslint-theme";
export default [...DiscourseThemeRecommended];
show original
cvx
14.Декабрь.2023 16:11:31
5
Хм, возможно, есть несовместимость между последней конфигурацией eslint и файлами ES6. Можете проверить, сохранятся ли ошибки после удаления суффиксов .es6?
pfaffman
14.Декабрь.2023 17:16:24
6
Вместо того чтобы читать это, я слепо скопировал всё из скелета темы.
Вы очень добры, что предполагаете, будто моя теория не была лишена оснований.
Ага. Значит, есть какая-то причина не делать глобальное переименование всех моих файлов es6 везде?
Мне кажется, я однажды это находил, что объясняет часть загадки.