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

test: remove yarn pack workaround #232

Merged
merged 10 commits into from
Aug 8, 2024
Merged

test: remove yarn pack workaround #232

merged 10 commits into from
Aug 8, 2024

Conversation

danielpza
Copy link
Member

@danielpza danielpza commented Aug 8, 2024

Remove the yarn pack workaround for testing. Using yarn link: or portal: protocol was not enough, we had to also link the ts-node dependency directly into the package node_modules so it could resolve it

@danielpza danielpza marked this pull request as ready for review August 8, 2024 04:36
@danielpza danielpza merged commit e421fa8 into master Aug 8, 2024
10 checks passed
@danielpza danielpza deleted the use-link branch August 8, 2024 04:39
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.

1 participant