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

Add local build caveats: name, version, and main in package.json #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chrisbolin
Copy link

I hit some issues trying this locally, so I figured I add some clarification to the docs.

The error thrown when version was not included:

UnhandledPromiseRejectionWarning: Error: Cannot find module '/tmp/tmp-build/packages/build-object-export-zRf/node_modules/LOCAL_PACKAGE/package.json'

And no line-number was called out.

@pastelsky pastelsky force-pushed the master branch 14 times, most recently from bd82b40 to ae5015f Compare October 17, 2020 11:47
@pastelsky
Copy link
Owner

I think we should handle cases where name / version might be undefined, if such a package.json file is considered a valid file.

I would consider this as a bug with our current implementation. Do you want to send a fix?

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