Skip to content

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

Open
chrisbolin wants to merge 1 commit intopastelsky:masterfrom
chrisbolin:patch-1
Open

Add local build caveats: name, version, and main in package.json#20
chrisbolin wants to merge 1 commit intopastelsky:masterfrom
chrisbolin:patch-1

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