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: ship less files to address TSC issues #35

Merged
merged 1 commit into from Jan 19, 2022
Merged

fix: ship less files to address TSC issues #35

merged 1 commit into from Jan 19, 2022

Conversation

jacob-ebey
Copy link
Contributor

Users have been running into issues caused by shipping the tsconfig.json in the package.

image

Users have been running into issues caused by shipping the tsconfig.json in the package.
@kentcdodds
Copy link

Ping @Gozala :) How can we help you get this merged/released?

@kiliman
Copy link

kiliman commented Jan 19, 2022

Hmm... since using web standards is a core foundation of Remix, perhaps it would be best to fork this and include it directly in Remix core. Doesn't seem like a lot of activity on a critical dependency.

Copy link
Contributor

@Gozala Gozala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine to me

@Gozala Gozala mentioned this pull request Jan 19, 2022
@Gozala Gozala merged commit 0651e62 into web-std:main Jan 19, 2022
@Gozala
Copy link
Contributor

Gozala commented Jan 19, 2022

Merged thanks for taking time to look into this and providing a fix.

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.

None yet

4 participants