search

fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.
https://demo.fullstack-development.com/
MIT License
91 stars 13 forks source link

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

Open sk1e opened 4 years ago

sk1e commented 4 years ago

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

in19farkt commented 4 years ago

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

noImplicitAny
strictNullChecks
strictFunctionTypes
strictBindCallApply
strictPropertyInitialization
noImplicitThis
alwaysStrict

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

in19farkt commented 4 years ago

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

    /* 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. */
Znack commented 4 years ago

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

sk1e commented 4 years ago

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

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

in19farkt commented 4 years ago

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

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

Safr commented 4 years ago

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