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

feat: Add .cjs, .mjs, .ts, .tsx, .jsx to default extensions #1110

Merged
merged 1 commit into from
May 18, 2019

Conversation

coreyfarrell
Copy link
Member

Fixes #1103


I don't want to pull TypeScript into the devDependencies so I'm not sure if this is going to get testing. Maybe we could put regular JS into files of these extensions?

The fact that extension was missing from the nyc report command is minor - only effected the nyc report --help output. Now that the default is non-empty it will matter. I'd like to look into creating groups of flags - for example a shared function which adds all flags that apply to test-exclude. That is not part of this.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 96.87% when pulling 86d46e9 on coreyfarrell:extension-list into fe3311b on istanbuljs:master.

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.

Expand default extension list
2 participants