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: handle JS files when allowJs and checkJs options are set to true #70

Closed
wants to merge 1 commit into from

Conversation

mkarajohn
Copy link

This pr allows the tool to handle JS files as well when the allowJs and checkJs are explicitly enabled in the user's config.

Resolves issues in the scenario when tsc-files runs as part of lint-staged over ts, tsx, js and jsx files.

If we do not handle JS files in this scenario, then they get passed along with the non-files arguments and we get the error about not being possible to mix -p along with files

@mkarajohn mkarajohn closed this May 8, 2024
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.

1 participant