Skip to content

Commit

Permalink
Issue #2: Dockerfile: install of dependencies in Docker
Browse files Browse the repository at this point in the history
The Dockerfile was incorrectly setting the list of
dependencies. It should be from the requirements.txt

This time the image is correctly built with the
dependencies, if new ones are added.
  • Loading branch information
marcellodesales committed Apr 26, 2019
1 parent 7823331 commit 8dfb934
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
FROM marcellodesales/github-enterprise-prereceive-hook-base
MAINTAINER Marcello_deSales@intuit.com

RUN \
apk add --no-cache py-pip && \
pip install yamllint pyyaml pyjavaproperties
RUN apk add --no-cache py-pip

ADD ./validate_config_files.py /home/git/test.git/hooks/pre-receive
COPY requirements.txt requirements.txt

RUN pip2 install -r requirements.txt

COPY ./validate_config_files.py /home/git/test.git/hooks/pre-receive

0 comments on commit 8dfb934

Please sign in to comment.