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

(Feature) Add dockerfile #1116

Merged
merged 1 commit into from
Aug 29, 2018
Merged

(Feature) Add dockerfile #1116

merged 1 commit into from
Aug 29, 2018

Conversation

fernandomg
Copy link
Contributor

Included Dockerfile contains travis, which may be helpful for running local test in a container before creating the PR.

Steps to use it:

  1. install docker https://docs.docker.com/install/#supported-platforms
  2. move to the project root
  3. docker build -t tw . (where tw is the name of the docker image created and . is the location of the Dockerfile)
  4. docker run -ti tw
  5. once the image has started, run travis compile > ci.sh
  6. edit ci.sh file and set the corresponding branch (search for branch word and replace: branch\=\'\' with branch\=\'master\')
  7. bash ci.sh

Docker build can receive three arguments author, project, branch.
Default values:

  • author: poanetwork
  • project: token-wizard
  • branch: master

So you can build a docker image pointing to a specific author/project/branch.
i.e.: docker build -t tw . --build-arg branch=2.0 will create the image pointing to branch 2.0


Important: Before running bash ci.sh you have to always find/replace the branch that the ci.sh is pointing to.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 2962

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 20.347%

Totals Coverage Status
Change from base Build 2950: 0.0%
Covered Lines: 834
Relevant Lines: 3594

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants