Skip to content

Use cross-spawn to help with Windows issues. #185

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

Closed
wants to merge 0 commits into from

Conversation

dominic-p
Copy link
Contributor

Fixes #180

This is a work in progress because I honestly couldn't figure out how to test it locally with the combination of tools that I don't really use: lerna + typescript + yarn.

Is there a contributing guide somewhere that I missed, or maybe you walk me through how to test locally?

@AJRdev
Copy link
Collaborator

AJRdev commented May 22, 2019

@dominic-p How to test locally is not really specific to this project, it is the same for every npm package development :-)

Usually you use a test application that you link to your local package ( this link might help : https://dev.to/vcarl/testing-npm-packages-before-publishing-h7o ) to test your new features.

You can also use in the repo the command yarn test to check if your changes didn't break any tests.

@dominic-p
Copy link
Contributor Author

Thanks for the tip. I'm familiar with testing basic npm packages, but I've never worked with lerna or yarn workspaces before. I'll give npm pack a try and let you know when I have some time.

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.

More Windows space in path issues
2 participants