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

Update the docker custom build #3807

Closed
wants to merge 2 commits into from
Closed

Update the docker custom build #3807

wants to merge 2 commits into from

Conversation

saurabharch
Copy link
Contributor

Update syntax command for docker COPY package.json and package*.json and n8n packages in the root working directory folder

Update syntax command for docker COPY package.json and package*.json  and n8n packages in the root working directory folder
@n8n-assistant n8n-assistant bot added the community Authored by a community member label Jul 30, 2022
Comment on lines +16 to +21
#COPY packages/cli/ ./packages/cli/
#COPY packages/core/ ./packages/core/
#COPY packages/design-system/ ./packages/design-system/
#COPY packages/editor-ui/ ./packages/editor-ui/
#COPY packages/nodes-base/ ./packages/nodes-base/
#COPY packages/workflow/ ./packages/workflow/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now including packages/node-dev

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please merge this request with #3846

@saurabharch saurabharch mentioned this pull request Aug 6, 2022
COPY packages/editor-ui/ ./packages/editor-ui/
COPY packages/nodes-base/ ./packages/nodes-base/
COPY packages/workflow/ ./packages/workflow/
COPY lerna.json .
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is no lerna.json in this repo anymore

@netroy
Copy link
Member

netroy commented Sep 2, 2022

This got addressed in #3973

@netroy netroy closed this Sep 2, 2022
@netroy
Copy link
Member

netroy commented Sep 2, 2022

@saurabharch Thanks for your contribution 💟

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants