-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PoC: type-checked rules #1
PoC: type-checked rules #1
Conversation
@@ -68,13 +68,28 @@ const config = { | |||
], | |||
parser: "@typescript-eslint/parser", | |||
parserOptions: { | |||
project: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
D'oh! I left this in, but meant to take it out. That's why on the stream I was still getting complaints about **/*.md/*.ts
files and type checking!
@JoshuaKGoldberg When working on this on stream, you asked yourself something in the lines of
I just disabled errors
Hope this helps you in understanding the choice that's been made in the past a bit better & makes the next steps easier |
WIP continuing from remix-run#6614. I'll split this up into a single PR per rule or group of rules to make it easier to review.
Note that remix-run#6331 already exists, so I don't need to redo work from it.