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

Remove TypeScript as an explicit peer dependency. #181

Merged
merged 2 commits into from
Feb 27, 2023
Merged

Conversation

Eyas
Copy link
Collaborator

@Eyas Eyas commented May 16, 2022

This is one potential fix to #180 that allows schema-dts to continue to be used
with import and any/no bundlers without increasing bundle size.

It is also possible to address #180 by simply asking users to save
schema-dts as a dev dependency. This is generally more correct for
typings, but runs the risk of leaving dangling imports to nonexistent
"schema-dts" js.

Eyas added 2 commits February 27, 2023 11:49
This is one potential fix to google#180 that allows schema-dts to continue to be used
with `import` and any/no bundlers without increasing bundle size.

It is also possible to address google#180 by simply asking users to save
schema-dts as a dev dependency. This is generally more correct for
typings, but runs the risk of leaving dangling imports to nonexistent
"schema-dts" js.

We _may_ want to add TypeScript as a dev dependency instead. I'm not
sure.
@Eyas Eyas merged commit dea38ff into google:main Feb 27, 2023
@Eyas Eyas deleted the ts-peer branch February 27, 2023 17:03
@Fonger
Copy link

Fonger commented Apr 13, 2023

Hi @Eyas any ETA about the new release?

I can't install this as dev dependencies because eslint complains about node/no-unpublished-import
I don't know why. I only use type import import type ... from 'schema-dts'.

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.

3 participants