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

Docker image creation #553

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Docker image creation #553

wants to merge 2 commits into from

Conversation

vikoivun
Copy link
Member

@vikoivun vikoivun commented Jul 3, 2018

No description provided.

@vikoivun vikoivun requested a review from jussiarpalahti July 3, 2018 16:29
@ghost ghost assigned vikoivun Jul 3, 2018
@ghost ghost added the in progress label Jul 3, 2018
@jussiarpalahti
Copy link
Member

Travis integration is needed.

Though I only added Travis in this branch https://github.com/City-of-Helsinki/servicemap/tree/feature/add-travis which is not merged yet (and isn't perhaps greatest implementation).

Otherwise looks good. Though I can't compute what work-in-π means.. 🤔

@vikoivun vikoivun changed the title [WIπ] Docker image creation [WIP] Docker image creation Jul 4, 2018
@vikoivun vikoivun changed the title [WIP] Docker image creation Docker image creation Jul 17, 2018
@vikoivun
Copy link
Member Author

Squashed into a single commit

* move everything wrt. configuration inside one method
* move everything wrt. server init inside another method
* separate client side and server side config in the config object
* use the separation to remove server stuff from client side
* install bower modules through `prepare` hook
* add empty configs for "dev" and "production" to signify validity
@vikoivun vikoivun force-pushed the feature/docker branch 2 times, most recently from a18cfa5 to 5f44922 Compare July 18, 2018 17:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants