Skip to content

Commit c805c7e

Browse files
committed
Refactor yaml
1 parent 650f712 commit c805c7e

1 file changed

Lines changed: 26 additions & 32 deletions

File tree

.github/workflows/basket-api.yml

Lines changed: 26 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -11,33 +11,31 @@ on:
1111
- src/Services/Basket/*
1212
- .github/workflows/basket-api.yml
1313
env:
14-
registryEndpoint: borjasanes
15-
services: basket-api
16-
image: basket.api
17-
# branch: $(echo ${GITHUB_REF#refs/heads/})
14+
REGISTRY_ENDPOINT: borjasanes
15+
SERVICES: basket-api
1816

1917
jobs:
2018
BuildContainersForPR_Linux:
2119
runs-on: ubuntu-16.04
2220
if: ${{ github.event_name == 'pull_request' }}
2321
steps:
24-
- name: Compose build ${{ env.services }}
25-
run: docker-compose build -f src/docker-compose.yml ${{ env.services }}
22+
- name: Compose build ${{ env.SERVICES }}
23+
run: docker-compose build -f src/docker-compose.yml ${{ env.SERVICES }}
2624
shell: bash
2725
env:
2826
TAG: ${{ env.branch }}
29-
REGISTRY: ${{ env.registryEndpoint }}
27+
REGISTRY: ${{ env.REGISTRY_ENDPOINT }}
3028
BuildContainersForPR_Windows:
3129
runs-on: windows-2019
3230
if: ${{ false }}
3331
# if: ${{ github.event_name == 'pull_request' }}
3432
steps:
35-
- name: Compose build ${{ env.services }}
36-
run: docker-compose build -f src/docker-compose.yml ${{ env.services }}
33+
- name: Compose build ${{ env.SERVICES }}
34+
run: docker-compose build -f src/docker-compose.yml ${{ env.SERVICES }}
3735
shell: pwsh
3836
env:
3937
TAG: ${{ env.branch }}
40-
REGISTRY: ${{ env.registryEndpoint }}
38+
REGISTRY: ${{ env.REGISTRY_ENDPOINT }}
4139
PLATFORM: win
4240
NODE_IMAGE: stefanscherer/node-windows:10
4341
BuildLinux:
@@ -53,52 +51,48 @@ jobs:
5351
registry:
5452
username: ${{ secrets.USERNAME }}
5553
password: ${{ secrets.PASSWORD }}
56-
# - name: Login to DockerHub Registry
57-
# run: echo ${{ secrets.PASSWORD }} | sudo docker login -u ${{ secrets.USERNAME }} --password-stdin
58-
# shell: bash
5954

60-
- name: Get branch name
55+
- name: Set branch name as env variable
6156
run: |
6257
currentbranch=$(echo ${GITHUB_REF##*/})
6358
echo "running on $currentbranch"
64-
echo "branch=$currentbranch" >> $GITHUB_ENV
65-
echo "TAG=$currentbranch" >> $GITHUB_ENV
59+
echo "BRANCH=$currentbranch" >> $GITHUB_ENV
6660
shell: bash
6761

68-
- name: Compose build ${{ env.services }}
62+
- name: Compose build ${{ env.SERVICES }}
6963
run: |
70-
sudo -E docker-compose build ${{ env.services }}
64+
sudo -E docker-compose build ${{ env.SERVICES }}
7165
docker images
7266
working-directory: ./src
7367
shell: bash
7468
env:
75-
# TAG: ${{ env.branch }}
76-
REGISTRY: ${{ env.registryEndpoint }}
69+
TAG: ${{ env.BRANCH }}
70+
REGISTRY: ${{ env.REGISTRY_ENDPOINT }}
7771

78-
- name: Compose push ${{ env.image }}
72+
- name: Compose push ${{ env.SERVICES }}
7973
run: |
8074
docker images
81-
sudo -E docker-compose push ${{ env.services }}
75+
sudo -E docker-compose push ${{ env.SERVICES }}
8276
working-directory: ./src
8377
shell: bash
8478
env:
85-
# TAG: ${{ env.branch }}
86-
REGISTRY: ${{ env.registryEndpoint }}
79+
TAG: ${{ env.BRANCH }}
80+
REGISTRY: ${{ env.REGISTRY_ENDPOINT }}
8781

8882
BuildWindows:
8983
runs-on: windows-2019
9084
if: ${{ false }}
9185
# if: ${{ github.event_name != 'pull_request' }}
9286
steps:
93-
- name: Compose build ${{ env.services }}
94-
run: docker-compose build -f src/docker-compose.yml ${{ env.services }}
87+
- name: Compose build ${{ env.SERVICES }}
88+
run: docker-compose build -f src/docker-compose.yml ${{ env.SERVICES }}
9589
shell: pwsh
9690
env:
97-
TAG: ${{ env.branch }}
98-
REGISTRY: ${{ env.registryEndpoint }}
99-
- name: Compose push ${{ env.image }}
100-
run: docker-compose push -f src/docker-compose.yml ${{ env.services }}'
91+
TAG: ${{ env.BRANCH }}
92+
REGISTRY: ${{ env.REGISTRY_ENDPOINT }}
93+
- name: Compose push ${{ env.SERVICES }}
94+
run: docker-compose push -f src/docker-compose.yml ${{ env.SERVICES }}'
10195
shell: pwsh
10296
env:
103-
TAG: ${{ env.branch }}
104-
REGISTRY: ${{ env.registryEndpoint }}
97+
TAG: ${{ env.BRANCH }}
98+
REGISTRY: ${{ env.REGISTRY_ENDPOINT }}

0 commit comments

Comments
 (0)