Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Bst dev server deploy configuration #422

Merged
merged 4 commits into from Dec 21, 2017
Merged

Bst dev server deploy configuration #422

merged 4 commits into from Dec 21, 2017

Conversation

jperata
Copy link
Contributor

@jperata jperata commented Dec 21, 2017

No description provided.

@coveralls
Copy link

coveralls commented Dec 21, 2017

Coverage Status

Changes Unknown when pulling faac69c on bstDevServer into ** on master**.

circle.yml Outdated
tag: /bst-dev-.*/
commands:
- docker login -e $DOCKER_EMAIL -u $DOCKER_USER -p $DOCKER_PASS
- docker build -f docker/Dockerfile -t bespoken/bst-dev:$CIRCLE_TAG .
Copy link
Member

Choose a reason for hiding this comment

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

I would keep the image name the same (bespoken/bst) - and use the tag to differentiate it (bst-dev-x or bst-server-dev-x). If you look in DockerHub, that is a cleaner way to organize images.

@coveralls
Copy link

coveralls commented Dec 21, 2017

Coverage Status

Changes Unknown when pulling c4a9ef8 on bstDevServer into ** on master**.

@jperata jperata merged commit 44f13a6 into master Dec 21, 2017
@jperata jperata deleted the bstDevServer branch December 21, 2017 20:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants