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

Docker compatibility using a Dockerfile #9

Merged
merged 2 commits into from
Oct 25, 2017
Merged

Conversation

random-robbie
Copy link
Contributor

@random-robbie random-robbie commented Oct 25, 2017

Working docker container.

docker build -t acstis .
docker run -it acstis:latest

Working docker container.

```
docker build -t acstis .
docker run -it acstis:latest

```
@tijme
Copy link
Owner

tijme commented Oct 25, 2017

Hi @random-robbie,

Really awesome to get a pull request from you 😄, thank you for contributing !

I'm not using Docker myself but I checked out the documentation and found a deprecation notice that applies to this pull request.

Maintainer Label
https://docs.docker.com/engine/reference/builder/#maintainer-deprecated

I think this should now be LABEL maintainer="txt3rob@gmail.com"

Please correct me if I'm wrong.

@tijme tijme added the Feature label Oct 25, 2017
@tijme tijme assigned tijme and unassigned tijme Oct 25, 2017
@tijme tijme requested review from tijme and removed request for tijme October 25, 2017 10:21
Copy link
Contributor Author

@random-robbie random-robbie left a comment

Choose a reason for hiding this comment

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

updated the label

@random-robbie
Copy link
Contributor Author

Hey dude,

Updated it now.

works wonders in docker.

@tijme tijme merged commit 394791e into tijme:master Oct 25, 2017
@tijme
Copy link
Owner

tijme commented Oct 25, 2017

Thanks again for your contribution. I just merged the Docker file to the master branch 😄 .

@random-robbie
Copy link
Contributor Author

👍 brilliant hopefully it helps others get it up and running.

@tijme tijme changed the title Create Dockerfile Docker compatibility using a Dockerfile Oct 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants