Replies: 15 comments 142 replies
-
May be fixed by #88 (
|
Beta Was this translation helpful? Give feedback.
-
Согласно документации - да |
Beta Was this translation helpful? Give feedback.
-
А styles кста хоть как придется вытащить)) Но да, хоть и есть регламент на "эталонные сегменты в shared", есть еще фактор специфичности проектов, где туда приходится самое разное складывать (router/styles/assets/...) И понятно, что использовать |
Beta Was this translation helpful? Give feedback.
-
|
Но кажется, можно решить общую проблему - стоит явно разделить ответственность правил
@Krakazybik @SQReder @tednaaa что думаете? 🤔 |
Beta Was this translation helpful? Give feedback.
-
|
с одной стороны да, много кейсов и стеков для построения |
Beta Was this translation helpful? Give feedback.
-
|
@Krakazybik Предлагаю все же остановиться на варианте с Т.к. мы действительно не сможем адаптировать конфиг под все возможные фреймворки, как бы не хотелось Но такая (лучше оформить отдельным PR - чисто для shared, чтоб не перемешивалось с pages) |
Beta Was this translation helpful? Give feedback.
-
ON-HOLD: Not enough details
|
Beta Was this translation helpful? Give feedback.
-
|
Пробовал и рутовый конфиг менять, и локальный - не помогает |
Beta Was this translation helpful? Give feedback.
-
|
Дело не только в кастомном сегменте components причем (здесь тоже должен триггериться на public-api и layers-slices)
|
Beta Was this translation helpful? Give feedback.
-
// @path package.json
{
"name": "angular-feature-sliced",
"version": "0.0.0",
"license": "MIT",
"sideEffects": false,
"scripts": {
"ng": "nx",
"postinstall": "node ./decorate-angular-cli.js && ngcc --properties es2015 browser module main",
"start": "nx serve",
"build": "nx build",
"lint": "eslint ./apps/angular-feature-sliced/src",
"test": "nx test"
},
"private": true,
"dependencies": {
"@angular/animations": "~13.1.0",
"@angular/cdk": "^13.1.1",
"@angular/common": "~13.1.0",
"@angular/compiler": "~13.1.0",
"@angular/core": "~13.1.0",
"@angular/forms": "~13.1.0",
"@angular/material": "^13.1.1",
"@angular/platform-browser": "~13.1.0",
"@angular/platform-browser-dynamic": "~13.1.0",
"@angular/router": "~13.1.0",
"@ngrx/component-store": "~13.0.0",
"@ngrx/effects": "~13.0.0",
"@ngrx/entity": "~13.0.0",
"@ngrx/router-store": "~13.0.0",
"@ngrx/store": "~13.0.0",
"@nrwl/angular": "13.4.1",
"rxjs": "~7.4.0",
"tslib": "^2.0.0",
"zone.js": "~0.11.4"
},
"devDependencies": {
"@angular-devkit/build-angular": "~13.1.0",
"@angular-eslint/eslint-plugin": "~13.0.1",
"@angular-eslint/eslint-plugin-template": "~13.0.1",
"@angular-eslint/template-parser": "~13.0.1",
"@angular/cli": "~13.1.0",
"@angular/compiler-cli": "~13.1.0",
"@angular/language-service": "~13.1.0",
"@feature-sliced/eslint-config": "0.1.0-beta",
"@feature-sliced/eslint-plugin-messages": "0.1.0-beta",
"@ngrx/schematics": "~13.0.0",
"@ngrx/store-devtools": "~13.0.0",
"@nrwl/cli": "13.4.1",
"@nrwl/cypress": "13.4.1",
"@nrwl/eslint-plugin-nx": "13.4.1",
"@nrwl/jest": "13.4.1",
"@nrwl/linter": "13.4.1",
"@nrwl/tao": "13.4.1",
"@nrwl/workspace": "13.4.1",
"@types/jest": "27.0.2",
"@types/node": "14.14.33",
"@typescript-eslint/eslint-plugin": "^5.3.1",
"@typescript-eslint/parser": "^5.3.1",
"cypress": "^9.1.0",
"eslint": "8.2.0",
"eslint-config-prettier": "8.1.0",
"eslint-import-resolver-typescript": "^2.5.0",
"eslint-plugin-boundaries": "^2.8.0",
"eslint-plugin-cypress": "^2.10.3",
"eslint-plugin-import": "^2.25.4",
"jasmine-marbles": "~0.9.1",
"jest": "27.2.3",
"jest-preset-angular": "11.0.0",
"prettier": "^2.3.1",
"ts-jest": "27.0.5",
"typescript": "~4.4.3"
}
}
|
Beta Was this translation helpful? Give feedback.
-
|
@Affiction А ты вроде внедрял у себя (если да , то пошарь пож конфиги гистом/веткой, а то воспроизвести у @Krakazybik не удается 🤔 ) |
Beta Was this translation helpful? Give feedback.
-
|
@Affiction ping) |
Beta Was this translation helpful? Give feedback.
-
May be fixed by #94 (
|
Beta Was this translation helpful? Give feedback.
-
|
А остальное +- норм работает
|
Beta Was this translation helpful? Give feedback.
-
|
И снова)
|
Beta Was this translation helpful? Give feedback.
-
|
@Krakazybik А тут есть затыки? Кажется, что если мы в |
Beta Was this translation helpful? Give feedback.
-
|
@Krakazybik ping) |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
MISC FeedbackNot only issues |
Beta Was this translation helpful? Give feedback.
-
|
На +- больших проектах - конфиг показывает себя тоже неплохо
|
Beta Was this translation helpful? Give feedback.
-
|
Тоже хорошо отлавливается
|
Beta Was this translation helpful? Give feedback.
-
|
И такое хоть "по идее" правильно, но тож забавляет)
|
Beta Was this translation helpful? Give feedback.
-
|
И даже такое бывает
Хотя справедливости ради - там +- локальная модель дергается 🤔 Тут в целом вопрос касаемо nested-pages
|
Beta Was this translation helpful? Give feedback.
-
May be fixed by #83 (
|
Beta Was this translation helpful? Give feedback.
-
|
@Krakazybik тут даже больше проблема - что PublicAPI вообще не задумываясь тригеррится на полное отстуствие реэкспортов на любой вложенности Если удастся ограничится тем, чтобы он ловил только для слайсов и сегментов - уже наполовину решило бы проблему
|
Beta Was this translation helpful? Give feedback.
-
|
Возможно, тут |
Beta Was this translation helpful? Give feedback.
-
|
@azinit в реферансах ссылка, но я что-то не найду такого там |
Beta Was this translation helpful? Give feedback.
-
|
Скрин - это уже с других проектов скрины в комментах) |
Beta Was this translation helpful? Give feedback.
-
|
Добавили лайт конфиг: |
Beta Was this translation helpful? Give feedback.
-
|
Problem: Too difficult "get started" |
Beta Was this translation helpful? Give feedback.
-
|
@Krakazybik JFYI |
Beta Was this translation helpful? Give feedback.
-
|
требует обсуждения, оставил мнение в #86 |
Beta Was this translation helpful? Give feedback.
-
|
@Krakazybik отписался в ишьюсе |
Beta Was this translation helpful? Give feedback.
-
|
Problem: Import order doesn't differentiate between regular imports and type imports I prefer my type imports to go last: import { startMediaListener } from './start-media-listener';
import type { Colors } from '../types';Rules: |
Beta Was this translation helpful? Give feedback.
-
|
Учитывая, что используется import/order - есть вероятность, что отдельно для |
Beta Was this translation helpful? Give feedback.
-
|
Мне кажется это вообще должно уйти в issues репы eslint-plugin-import, а не сюда |
Beta Was this translation helpful? Give feedback.
-
May be fixed by #76 (
|
Beta Was this translation helpful? Give feedback.
-
|
благодаря авто-импортам IDE получаем такой эффект)
UPD. в целом много ошибок получаем из за |
Beta Was this translation helpful? Give feedback.
-
|
Соль как раз в том, чтобы не поощрать автоимпорты "вслепую" |
Beta Was this translation helpful? Give feedback.
-
|
Кастомизацию помощнее тоже думали завозить, но и строгость от методологии тоже должна быть какая-то 🤔 |
Beta Was this translation helpful? Give feedback.
-
|
Я за строгость =) |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
May be fixed by #80 (
|
Beta Was this translation helpful? Give feedback.
-
|
May be will fixed by |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
May be fixed by #79 (
|
Beta Was this translation helpful? Give feedback.
-
|
@Krakazybik дополни если считаешь нужным, но думаю ты +- такого же мнения |
Beta Was this translation helpful? Give feedback.
-
Об этом как-то изначально и не подумали.
По сути, то что сейчас, это некая фикция =) По фатку оно import-order, хоть и процессор его выплёвывает в другом виде
Для исправления в принципе грохнуть ruleId из объектов, но только вот оно закэшировалось, и не известно сколько будет висеть у юзера |
Beta Was this translation helpful? Give feedback.
-
|
Проблема снова повторяется, но уже у другого человека @Krakazybik кажется, надо фиксить)
|
Beta Was this translation helpful? Give feedback.
-
|
Также из-за этой разнородицы в ruleIds не всегда (не у всех*) удается задизейблить локально действие правил 🤔
|
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
May be fixed by #82 (
|
Beta Was this translation helpful? Give feedback.
-
|
Дело вкуса, но для меня лучший вариант, верх - |
Beta Was this translation helpful? Give feedback.
-
Я обычно делаю как раз так ( |
Beta Was this translation helpful? Give feedback.
-
|
Нууу, пока занесли в Если зайдет людям - то при мажорном релизе - схлопнем в одно 🤔 |
Beta Was this translation helpful? Give feedback.
-
|
Спасибо всем, что высказались! |
Beta Was this translation helpful? Give feedback.
-
|
Я считаю, что порядок должен быть
Обратный порядок же превратит импорты в лютую кашу. import axios from 'axios' // Самый топ-левел (Возьмём его за 0)
import { Foo } from '@/widgets/foo' // Виджеты (4)
import { Bar } from '@/features/bar' // Фичи (3)
import { Bar } from '@/entities/baz' // Сущности (2)
import { Button } from '@/shared/ui' // Shared (1)
import { Smth } from './ui/Smth' // Локальное от страницы (5)И шо это за дичь)) Как с американским форматом дат история выходит Если уж кому-то так хочется - то можно это сделать опцией, но лично я категорически против принимать это за дефолтное поведение, это совершенно не логично. |
Beta Was this translation helpful? Give feedback.
-
May be fixed by #95 (
|
Beta Was this translation helpful? Give feedback.
-
Заманчивое предложение)) Но аргументы тоже понимаю |
Beta Was this translation helpful? Give feedback.
-
@Krakazybik кста вот уже первый костыль в голову пришел - для кастомных сегментов можно сделать, чтобы "эталонные сегменты" норм выглядели, а кастомные обязательно черту чтоб содержали)) " |
Beta Was this translation helpful? Give feedback.
-
|
|
Beta Was this translation helpful? Give feedback.
-
|
Предлагаю все же остановиться на варианте с Т.к. мы действительно не сможем адаптировать конфиг под все возможные фреймворки, как бы не хотелось Но такая (лучше оформить отдельным PR от shared, и учитывая, что у нас щас есть эксперименты с |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
May be fixed by #82
|
Beta Was this translation helpful? Give feedback.
-
@SQReder что думаешь по этому поводу? Или в твоем видении, все же исчезающе редко будет случаев, когда на один файл используется по одному модулю с одного слоя? |
Beta Was this translation helpful? Give feedback.
-
|
предлагаю, для страждущих, через кастомное подключение сделать возможность 'newlines-between': 'always', как отдельное правило внутри import-order, что только конфиг немного страшненький будет =)) |
Beta Was this translation helpful? Give feedback.
-
|
Ну вот в том и проблема, что чтобы подключить одну малюсенькую опцию с newlines-between, придется полностью "эджектить" конфиг для import/order) |
Beta Was this translation helpful? Give feedback.
-
|
смотря на сколько популярна будет данная опция =) Если оно многим надо - то почему бы в основной её не засунуть, если будет пользоваться полтора человека - ну что тут сказать, пусть страдают =)))) да и не такая большая проблема скопипастить будет |
Beta Was this translation helpful? Give feedback.
-
В вырожденном случае - по одному импорту со слоя может смотреться избыточно. Но на большом количестве это окупает себя |
Beta Was this translation helpful? Give feedback.
-
|
Problem: for ui segment error is not occured
Rules: public-api |
Beta Was this translation helpful? Give feedback.
-
|
@tednaaa Ждем контекста для репродьюса в общем) Иначе не можем что-либо делать по баге |
Beta Was this translation helpful? Give feedback.
-
|
такс, сори за ожидание, сейчас попытаюсь побольше контекста дать
единственное что от себя могу добавить, это то что да, трабл может быть в самом vscode хотя если запустить нпм скрипт
единственная пока наиболее верная догадка, это то что имеем доп. вложенность upd: попробовал поменять регистр - к сожалению тоже не помогло |
Beta Was this translation helpful? Give feedback.
-
|
@tednaaa Спасибо, а можешь пож приложить еще:
|
Beta Was this translation helpful? Give feedback.
-
@tednaaa это очень бы помогло |
Beta Was this translation helpful? Give feedback.
-
через звездочку кстати тоже не работает
module.exports = {
extends: ['alloy', 'alloy/react', '@feature-sliced'],
settings: {
react: {
version: 'detect',
},
},
plugins: ['jest', '@feature-sliced/eslint-plugin-messages'],
processor: '@feature-sliced/messages/fs',
env: {
'jest/globals': true,
},
};
{
"compilerOptions": {
"baseUrl": "./src",
"paths": {
"@/*": ["./*"]
}
},
"typeAcquisition": {
"include": ["jest"]
}
} |
Beta Was this translation helpful? Give feedback.
-
|
Я очень изменяюсь за не самый умный вопрос, но данный конфиг всё ещё "живой"? Я заметил, что последний коммит был полгода назад (с сегментом assets), а последняя версия в нпм была полтора года назад. Последних изменений как раз и нехватает. :/ |
Beta Was this translation helpful? Give feedback.
-
|
Приветствую!) Сейчас близится новый крупный релиз стейбла, где в том числе будут пофикшены основные баги по еслинт конфигу, чтобы сделать его стабильным в применении Если же из существующих багов есть критичные или не хватает по вашему опыту - дайте знать об этом в ишьюсах) |
Beta Was this translation helpful? Give feedback.
-
|
Понятно. На данный момент я тупо скопировал выши конфиги к себе и кастомизировал как мне надо, но буду иметь ввиду, спасибо.) |
Beta Was this translation helpful? Give feedback.
-
You can leave here your feedback and issues about
0.1.0-beta🍰Some issues were considered here
Beta Was this translation helpful? Give feedback.
All reactions