Skip to content
This repository has been archived by the owner on Nov 10, 2021. It is now read-only.

馃攢 Use production build for deploy #27

Closed
wants to merge 1 commit into from
Closed

馃攢 Use production build for deploy #27

wants to merge 1 commit into from

Conversation

inDream
Copy link

@inDream inDream commented Jun 11, 2018

Connects likecoin#5

@inDream inDream added the review label Jun 11, 2018
@inDream inDream requested a review from rickmak June 11, 2018 12:10
@ghost ghost assigned inDream Jun 11, 2018
rickmak
rickmak approved these changes Jun 12, 2018
View reviewed changes
Dockerfile.production
WORKDIR /usr/src/app

# TODO: make it multi-build and only run npm install production
Copy link
Member

Choose a reason for hiding this comment

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

Let convert it to issue.

@rickmak
Copy link
Member

rickmak commented Jun 12, 2018

Merge via 0d6381b

@rickmak rickmak closed this Jun 12, 2018
@ghost ghost removed the review label Jun 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rickmak rickmak rickmak approved these changes

Assignees

@inDream inDream

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@inDream @rickmak