valantic / vue-template

A custom Vue.js boilerplate based on webpack.
MIT License
7 stars 4 forks source link

TS config improvals #335

Closed patric-eberle closed 10 months ago

patric-eberle commented 11 months ago

Pull request

This PR checks and extends the TS config file according to the recommendations from this video: https://www.youtube.com/watch?v=eJXVEju3XLM

I decided not to use noUncheckedIndexAccess thought, because it seems to conflict with some of our code (e.g. in Vue templates). But tried to remove shown issues where ever possible.

Ticket

331

Browser testing

Checklist

Review/Test checklist

mob8607 commented 10 months ago

There is an existing PR https://github.com/valantic/vue-template/pull/332/files for this topic. Please check the differences and merge them together.

patric-eberle commented 10 months ago

@mob8607 I compared the two branches and merged the changes from the first one if missing or better solution. I also applied the requested changes from the first branch. Please check again.

patric-eberle commented 10 months ago

@BeneRichi can you please do an additional review of this PR. Mathias is blocked by projects.