Skip to content

Comments

Fix build and prepare step#3

Merged
SleeplessByte merged 1 commit intomasterfrom
hotfix/docker
Jun 6, 2019
Merged

Fix build and prepare step#3
SleeplessByte merged 1 commit intomasterfrom
hotfix/docker

Conversation

@SleeplessByte
Copy link
Member

yarn install will install devDependencies

  • additionally it runs yarn prepublish (because that's how it works)
  • this in turn runs yarn test
  • this in turn runs yarn build (what we want) and jest (to validate)

This means that the tests MUST be present when building the image. They are not copied over, so this is not an issues.

Additionally, this creates a new production_node_modules folder with the right flags set, so that the final image node_modules is extremely small.

yarn install will install devdependencies

- additionally it runs yarn prepublish (because that's how it works)
- this in turn runs yarn test
- this in turn runs yarn build (what we want) and jest (to validate)

This means that the tests MUST be present when building the image. They are *not* copied over, so this is not an issues.

Additionally, this creates a new production_node_modules folder with the right flags set, so that the final image node_modules is extremely small.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working x:size/massive Massive amount of work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant