evoytenkoapps / angular-best-practices Goto Github PK

я конечно дико извиняюсь но я не вижу описания плюшек от соблюдения правил которые содержатся в readme (пример: don't use any, почему я не должен его использовать? как я должен понять что это уместно или нет? ответа не дано), еще и оригинал на английском без перевода (стоит подумать о тех кому сложно английский читать), у нодовских бестпрактисов к примеру описано что мы получаем взамен соблюдения правилам и есть перевод

было бы неплохо иметь в начале README список всех разделов.. а то все время приходится скролить и искать нужный раздел

Мне кажеться более правильный вариант будет использовать ReplaySubject для отписок.
Предлагаю
const destroy$: Subject = new Subject();
Заменить на
const destroy$ = new ReplaySubject(1);
https://medium.com/ngx/why-do-you-need-unsubscribe-ee0c62b5d21f

Или использовать вариант сервиса например как тут https://github.com/TinkoffCreditSystems/taiga-ui/blob/main/projects/cdk/services/destroy.service.ts#L8.

Зависимости
Устанавливаем зависимости по согласованию

Зависимости должны устанавливаться с возможностью обновления только patch версии. Для этого указываем символ ~ и номер версии в package.json.

это вообще плохой тон и приводит к несостыковке версий между машинами... В "лучших практиках" применимо понятие "фиксированная версия" без всяких тильд и домиков и прочих динамических версий! чтобы при npm install всегда устанавливалась только та версия пакета и их завасимостей, которая прописана в package.json, а не та, которая может случайно прилететь патчем и никто и не поймет что произошло на самом деле если что-либо отвалится!

"раках" -> "рамках"

Tslint is deprecated and must be replaced with eslint. Especially after migration to angular v12+.

Добрый день.
Как я понимаю здесь перепутано "Неверный код:" с "Хороший код:"
Для того, чтобы отписаться, оператор takeUntil (.....)должен стоять рядом subscribe(), а не вверху.
Неверный код:

this.uneditable$ = this.api.getClassifiersTopShowOnGUI().pipe(
   map((response) => response.result),
   takeUntil(this.destroy$)
 ).subscribe();

Хороший код:

    this.uneditable$ = this.api.getClassifiersTopShowOnGUI().pipe(
      takeUntil(this.destroy$)
      map((response) => response.result)
    ).subscribe();