Skip to content

(fix): check for JSX extension for entry files#474

Merged
swyxio merged 1 commit intojaredpalmer:masterfrom
agilgur5:fix-jsx-entry
Feb 2, 2020
Merged

(fix): check for JSX extension for entry files#474
swyxio merged 1 commit intojaredpalmer:masterfrom
agilgur5:fix-jsx-entry

Conversation

@agilgur5
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator Author

agilgur5 commented Feb 2, 2020

Let's give this a try...

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

@allcontributors
Copy link
Copy Markdown
Contributor

@agilgur5

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

@agilgur5
Copy link
Copy Markdown
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