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(ts): Add types to exports fields #1628

Closed
wants to merge 1 commit into from

Conversation

chentsulin
Copy link

@chentsulin chentsulin commented Jun 14, 2022

Description

Fix #1602

See https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#package-json-exports-imports-and-self-referencing for more details.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@vercel
Copy link

vercel bot commented Jun 14, 2022

@chentsulin is attempting to deploy a commit to the The Guild Team on Vercel.

A member of the Team first needs to authorize it.

@mikaelkaron
Copy link

Any reason this can't be merged? Also having typescript errors when using esm.

@chentsulin
Copy link
Author

@Urigo Is there any chance to move this forward? This blocks people using TS ESM with graphql-scalars.

@vercel
Copy link

vercel bot commented Aug 19, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
graphql-scalars ❌ Failed (Inspect) Aug 19, 2022 at 4:32AM (UTC)

@ardatan
Copy link
Collaborator

ardatan commented Oct 10, 2022

Done with 5bf309c
Thanks!

@ardatan ardatan closed this Oct 10, 2022
@chentsulin chentsulin deleted the patch-1 branch October 11, 2022 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants