blitz-js / blitz-js/legacy-framework

Use strict TypeScript option for new apps

Open
#39 2 comments 3 reactions 0 assignees View on GitHub
kind/feature-change legacy-framework status/icebox
Dominant language
JavaScript
Stars
3
Forks
2
PR merge metrics
No merged PRs in 30d

Description

### What do you want and why?

We should turn on the tsconfig `strict` mode by default. `strict` mode allows the static type checker to catch even more potential problems!

### Possible implementation(s)

Set `strict` to `true` in https://github.com/blitz-js/blitz/blob/canary/packages/generator/templates/app/tsconfig.json

### Additional context

Searching for this, it seems that there was a PR to address some strict mode issues https://github.com/blitz-js/blitz/pull/970 but I don't see one where we turn it on by default.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.