fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.

Home Page:https://demo.fullstack-development.com/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

установить флаг strict: true в tsconfig.json

sk1e opened this issue · comments

почему strictFunctionTypes сейчас выставлен в false?

там вообще нужно убрать эти флаги

noImplicitAny
strictNullChecks
strictFunctionTypes
strictBindCallApply
strictPropertyInitialization
noImplicitThis
alwaysStrict

и поставить один, который всё это объединяет"strict": true,

И еще эти нужно включить (если они не включены)

    /* Additional Checks */
    "noUnusedLocals": true,                /* Report errors on unused locals. */
    "noUnusedParameters": true,            /* Report errors on unused parameters. */
    "noImplicitReturns": true,             /* Report error when not all code paths in function return a value. */
    "noFallthroughCasesInSwitch": true,    /* Report errors for fallthrough cases in switch statement. */

Поправил название для "strict": true, @sk1e норм же?

Поправил название для "strict": true, @sk1e норм же?

да, я просто выяснить хотел почему именно strictFunctionTypes в false выставлен был. Выглядит преднамеренно

да, я просто выяснить хотел почему именно strictFunctionTypes в false выставлен был. Выглядит преднамеренно

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

Лучше добавлять флаги отдельно, так мы внесем ясность что происходит, и с strict:true есть проблемы, иногда мы знаем больше о коде , чем тс, но так как все в строгом режиме он нам будет ошибки сыпать