Skip to content
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

Fix directive check in no-empty-file for TypeScript parser #2180

Merged
merged 4 commits into from
Jul 25, 2023

Conversation

mcous
Copy link
Contributor

@mcous mcous commented Jul 23, 2023

Fixes #2175

To ensure the fix is valid, I updated the @typescript-eslint/parser dev dependency and added a few simple test.typescript cases to no-empty-file tests. I attempted to update @typescript-eslint/parser to v6, but it caused the angular integration test to fail, so I reverted that change. Given that this is still being tested on the v5 parser, the tests I added don't actually do much yet, but will provide checks once the parser is upgraded.

Please let me know if there's any additional testing I can / should add to this PR!

@mcous mcous force-pushed the fix-no-empty-file branch from 4952099 to 362255e Compare July 23, 2023 23:38
@mcous mcous changed the title Fix directive check in no-empty-file for TypeScript parser Fix directive check in no-empty-file for TypeScript parser Jul 23, 2023
@fisker fisker self-requested a review July 24, 2023 00:13
rules/no-empty-file.js Outdated Show resolved Hide resolved
@fisker fisker self-assigned this Jul 25, 2023
@sindresorhus sindresorhus merged commit 1bb9ce8 into sindresorhus:main Jul 25, 2023
@mcous mcous deleted the fix-no-empty-file branch July 25, 2023 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

no-empty-file always false positives in @typescript-eslint/parser@6
3 participants