Explicit tsconfig target & module

[adamalfredsson]

These are the default settings for target & module. It seems conventional to make this explicit. ChatGPT lists some reasons:

1. Clarity: Being explicit about your configuration can make it clearer for anyone reading the configuration, especially for those who might not be aware of the defaults. It also leaves no room for doubt about what values are being used.
2. Stability: If TypeScript ever changes its default settings in a future version (which is unlikely but not impossible), your project won't be unexpectedly affected because you've explicitly set the values you want.
3. Self-Documentation: By explicitly setting these configurations, developers don't need to look up the defaults, reducing the potential for errors or misunderstandings. This can be particularly beneficial for less experienced developers or those new to TypeScript.
4. Explicit Migrations: If you decide to change these settings in the future, having them already present makes it clear that you're intentionally changing a setting rather than just adding one.

---

Tested on devices

• Desktop 💻
• Mobile 📱

Tests

• All tests are running ✔️
• Test are updated 🧪
• Code Review 👩‍💻
• QA 👌

Checkpoints

Check these to flag for a more thurough review, as they could be potentially breaking changes

• Packages updated
• Other infrastructure updated (such as node version or similar)

⏲️ Time spent on CR:

⏲️ Time spent on QA:

[fellmirr]

@adamalfredsson To be clear, these are the values default resolve to?

[adamalfredsson]

Yes!

https://www.typescriptlang.org/tsconfig#target

https://www.typescriptlang.org/tsconfig#module