test: remove type in errors - #2966
Conversation
|
|
FYI: Lint errors fixed in #2981, so I updated your branch with that. |
|
Just a heads-up: there are other test cases in the repository aren’t verifying the location info. |
FloEdelmann
left a comment
There was a problem hiding this comment.
Thank you! 🙂
Just a heads-up: there are other test cases in the repository aren’t verifying the location info.
Yeah, I know. I just didn't want to make the tests weaker by removing type and not adding location. Ideally we'd add the location everywhere but that's clearly out of scope for this PR 😅
Related:
LintMessage#nodeTypeandTestCaseError#typeeslint/eslint#19029