Closed rcantin-w closed 1 day ago
Size Change: +19 B (0%)
Total Size: 978 kB
Filename | Size | Change | |
---|---|---|---|
content/webapp/.next/static/chunks/4621-6389983752d98bfa.js |
0 B | -5.11 kB (removed) | 🏆 |
identity/webapp/.next/static/chunks/pages/index-bc73c6d82c3f3d85.js |
0 B | -12.1 kB (removed) | 🏆 |
content/webapp/.next/static/chunks/4621-d854f758923784b3.js |
5.11 kB | +5.11 kB (new file) | 🆕 |
identity/webapp/.next/static/chunks/pages/index-744b70e8931f1e55.js |
12.1 kB | +12.1 kB (new file) | 🆕 |
Who is this for?
10844 . Best practices and maintenance.
What does it change?
tsconfig-base.json
file that gets extended in all workspace subrepos, and overwritten there if necessarycompilerOptions
,include
andexclude
tsconfig-base.json
to Docker builds so it can be referencedHow did I go about it
I did an audit and then tried to dig deeper on each option here, so have a nosy at your leisure!
I then offered it up to everyone over the last week and ran through it with David in the last FE catch-up. I am far from an expert here, learned a lot from doing it, so will take any suggestions.
I also wonder if it'd be useful to add comments to the files, so let me know what kind of comments you'd find useful if so.
Findings
tsconfig
in the way it sees fits, which not everyone is happy about but is the way it is. In our case it:moduleResolution
benode
, which I wanted to move away from since apparently that meansnode10
and older(!). There's no getting around it though. So I movedmodule
andmoduleResolution
over to individualtsconfig
files instead of the main one.es6
as we do not support IE anymore.strict: true
in the future, which I'll create a new ticket for.TODO: Create tickets for