Skip to content

Conversation

kinyoklion
Copy link
Member

@kinyoklion kinyoklion commented May 16, 2023

I ended up using the release-please extra files instead of the .default. The edge environment doesn't seem to agree with TS expectations here, so .default would violate the local typings.

The test file still loads from pacakge.json, which passes here, and should pass as the release PRs update things. If the module name changed, it would fail, which is good actually.

After this is merged we can check the release PR.

@kinyoklion kinyoklion marked this pull request as ready for review May 16, 2023 16:17
@kinyoklion kinyoklion requested a review from ldhenry May 16, 2023 16:18
Copy link
Contributor

@ldhenry ldhenry left a comment

Choose a reason for hiding this comment

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

Thanks!

@kinyoklion kinyoklion merged commit 9a5dcb8 into main May 16, 2023
@kinyoklion kinyoklion deleted the rlamb/importing-packages-vercel branch May 16, 2023 17:16
@github-actions github-actions bot mentioned this pull request May 16, 2023
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.

2 participants