Skip to content

Commit

Permalink
Merge pull request #78 from chrisweb/preview
Browse files Browse the repository at this point in the history
Preview to main
  • Loading branch information
chrisweb authored Nov 27, 2024
2 parents a21d21c + 7318a54 commit 0238cd9
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
@@ -291,7 +291,7 @@ What this override does:

### Option 2: everything that is in option 1 + typescript code rules

I like option 2 best as it adds a lot of good rules that will check your typescript code and give you feedback if needed. If you too decide to use this option, but it finds too many problems in your code, then maybe set the rules to **warn** instead of **error** until you have time to fix problems that get reported. Later you can switch back to **error** to enforce the rules (or you might want to disactivate some of the rules completly)
I like option 2 best as it adds a lot of good rules that will check your typescript code and give you feedback if needed. If you too decide to use this option, but it finds too many problems in your code, then maybe set the rules to **warn** instead of **error** until you have time to fix problems that get reported. Later you can switch back to **error** to enforce the rules (or you might want to deactivate some of the rules completely)

If you chose option 2 (the stricter version with more typescript-related rules), we first need to install the additional **@typescript-eslint/eslint-plugin** package, but to ensure that this package uses the same version as the **@typescript-eslint/parser** package, I recommend installing both, like so:

0 comments on commit 0238cd9

Please sign in to comment.