Merge pull request #1 from skilld-labs/use-cmd

Use CMD instead of ENTRYPOINT

Author: andriyun

Dockerfile

@@ -1,21 +1,28 @@
 FROM alpine:edge
-MAINTAINER Harry Walter
+
 MAINTAINER Andriy Yun <andriy.yun@gmail.com>
+MAINTAINER Andy Postnikov <apostnikov@gmail.com>
 
 RUN echo "@testing http://dl-cdn.alpinelinux.org/alpine/edge/testing" >> /etc/apk/repositories \
-    && apk add --update php7@testing php7-xml@testing php7-phar@testing \
-    php7-openssl@testing php7-mbstring@testing php7-json@testing php7-ctype@testing \
-    curl patch \
-    && rm -fr /var/cache/apk/* \
-    && ln -s /usr/bin/php7 /usr/bin/php \
-    && curl -sS https://getcomposer.org/installer | php -- --filename=composer --install-dir=/usr/bin
-
-RUN composer global require drupal/coder \
-&& ln -s /root/.composer/vendor/squizlabs/php_codesniffer/scripts/phpcs /usr/bin/phpcs \
-&& ln -s /root/.composer/vendor/squizlabs/php_codesniffer/scripts/phpcbf /usr/bin/phpcbf \
-&& ln -s /root/.composer/vendor/drupal/coder/coder_sniffer/Drupal /root/.composer/vendor/squizlabs/php_codesniffer/CodeSniffer/Standards/Drupal
+  && apk add --no-cache \
+  php7@testing \
+  php7-xml@testing \
+  php7-phar@testing \
+  php7-openssl@testing \
+  php7-mbstring@testing \
+  php7-json@testing \
+  php7-ctype@testing \
+  curl \
+  patch \
+  && ln -s /usr/bin/php7 /usr/bin/php \
+  && curl -sS https://getcomposer.org/installer | php -- --filename=composer --install-dir=/usr/bin \
+  && composer global require drupal/coder \
+  && ln -s /root/.composer/vendor/squizlabs/php_codesniffer/scripts/phpcs /usr/bin/phpcs \
+  && ln -s /root/.composer/vendor/squizlabs/php_codesniffer/scripts/phpcbf /usr/bin/phpcbf \
+  && ln -s /root/.composer/vendor/drupal/coder/coder_sniffer/Drupal /root/.composer/vendor/squizlabs/php_codesniffer/CodeSniffer/Standards/Drupal \
+  && ln -s /root/.composer/vendor/drupal/coder/coder_sniffer/DrupalPractice /root/.composer/vendor/squizlabs/php_codesniffer/CodeSniffer/Standards/DrupalPractice
 
 VOLUME /work
 WORKDIR /work
 
-ENTRYPOINT ["phpcs", "--standard=Drupal"]
+CMD ["phpcs", "--standard=Drupal", "."]

README.md

@@ -1,16 +1,24 @@
 # docker-phpcs-drupal
 
 ## How to use
-Go to needed module or file
+Go to needed directory with code
 ```
 cd /path/to/check
 ```
 By default we call phpcs from container
 ```
-docker run --rm -v $(pwd):/work docker-phpcs-drupal ./
+docker run --rm -v $(pwd):/work skilldlabs/docker-phpcs-drupal
 ```
 
-To call phpcbf you need rewrite entrypoint directive
+To call phpcbf you need to override default command
 ```
-docker run --rm -v $(pwd):/work --entrypoint=phpcbf docker-phpcs-drupal --standard=Drupal ./
+docker run --rm -v $(pwd):/work skilldlabs/docker-phpcs-drupal phpcbf --standard=Drupal .
 ```
+
+## Shortening of container name
+
+```
+docker tag skilldlabs/docker-phpcs-drupal phpcs
+```
+
+then you can use `docker run --rm -v $(pwd):/work phpcs`