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): check for JSX extension for entry files #474

Merged
merged 1 commit into from
Feb 2, 2020

Conversation

agilgur5
Copy link
Collaborator

@agilgur5 agilgur5 commented Jan 30, 2020

  • previously, TS, TSX, and JS were checked, but not JSX

Fixes #143 . Credits to @sisp for finding this and providing the suggested solution there. A PR hadn't ever been made and that issue's a bit old; this is related to #473 and #443 so figured it should be done sooner rather than later.

@agilgur5 agilgur5 changed the title (fix): JSX extension should be valid for entry files (fix): check for JSX extension for entry files Jan 30, 2020
- previously, TS, TSX, and JS were checked, but not JSX
@agilgur5
Copy link
Collaborator Author

Recurrence of the missing .d.ts file on Ubuntu in CI here: https://github.com/jaredpalmer/tsdx/pull/474/checks?check_run_id=418238622

@swyxio swyxio merged commit 67a07bd into jaredpalmer:master Feb 2, 2020
@agilgur5
Copy link
Collaborator Author

agilgur5 commented Feb 2, 2020

Let's give this a try...

@all-contributors please add @sisp for bug, code

@allcontributors
Copy link
Contributor

@agilgur5

I've put up a pull request to add @sisp! 🎉

@agilgur5
Copy link
Collaborator Author

agilgur5 commented Feb 2, 2020

Cool, guess you don't need to be a collaborator to ask the bot!

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.

Support Rollup input extension .jsx
2 participants