Merge pull request #298 from ssssank/fix_docker_compose

[#284] Fix docker-compose. Use override

.github/workflows/Docker.yml

@@ -8,42 +8,12 @@ on:
     branches:
       - main
 
-env:
-  # TODO: Change variable to your registry.
-  REGISTRY_NAME: hexletbasics
-
 jobs:
-  build-test:
+  main:
     runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - name: Run tests
-        run: |
-          docker-compose --file docker-compose.test.yml build
-          docker-compose --file docker-compose.test.yml run sut
 
-  build-push-notify:
-    runs-on: ubuntu-latest
-    if: ${{ github.event_name == 'push' }}
-    needs: build-test
     steps:
-      - uses: actions/checkout@v2
-      - name: Set repository name
-        run: |
-          echo "REPOSITORY_NAME=$(echo "$GITHUB_REPOSITORY" | awk -F / '{print $2}')" >> $GITHUB_ENV
-      - name: Build and push into docker hub
-        uses: docker/build-push-action@v1.1.0
+      - uses: hexlet-basics/exercises-action@release
         with:
-          repository: ${{ env.REGISTRY_NAME }}/${{ env.REPOSITORY_NAME }}
-          username: ${{ secrets.DOCKER_USERNAME }}
-          password: ${{ secrets.DOCKER_PASSWORD }}
-          tags: latest
-      - name: Set fail Slack env vars
-        if: ${{ failure() }}
-        run: |
-          echo "SLACK_COLOR=#a30200" >> $GITHUB_ENV
-      - name: Send notification into Slack
-        if: ${{ always() }}
-        uses: rtCamp/action-slack-notify@v2.0.0
-        env:
-          SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
+          DOCKER_USERNAME: ${{ secrets.DOCKER_USERNAME }}
+          DOCKER_PASSWORD: ${{ secrets.DOCKER_PASSWORD }}

Makefile

@@ -11,6 +11,9 @@ compose-build:
 compose-install:
 	docker-compose run exercises composer install
 
+compose-code-lint:
+	docker-compose run exercises make code-lint
+
 code-lint:
 	composer exec --verbose phpcs -- modules
 
@@ -22,3 +25,13 @@ compose-bash:
 
 compose-test:
 	docker-compose run exercises make test
+
+compose-description-lint:
+	docker-compose run exercises make description-lint
+
+compose-schema-validate:
+	docker-compose run exercises make schema-validate
+
+ci-check:
+	docker-compose --file docker-compose.yml build
+	docker-compose --file docker-compose.yml up --abort-on-container-exit

README.md

@@ -10,17 +10,20 @@
 
 ```sh
 # setup
-$ make
+make
 # run
-$ make compose
+make compose
 
 # run tests
-$ make compose-test
+make compose-test
 
 # run linters and validators
-$ make compose-code-lint
-$ make compose-description-lint
-$ make compose-schema-validate
+make compose-code-lint
+make compose-description-lint
+make compose-schema-validate
+
+
+make ci-check # run CI checks witn docker-compose
 ```
 
 В тестах используется [phpower](https://github.com/ngyuki/phpower). Внимательнее при работе с ассертами.

docker-compose.override.yml

@@ -0,0 +1,8 @@
+---
+
+version: '3'
+
+services:
+  exercises:
+    volumes:
+      - .:/exercises-php

docker-compose.yml

@@ -5,6 +5,5 @@ version: '3'
 services:
   exercises:
     build: .
-    volumes:
-      - .:/exercises-php
+    image: hexletbasics/exercises-php:cached
     command: make check