Skip to content

Commit

Permalink
Merge pull request #67 from BaseballCardTracker/29-dependencies-in-do…
Browse files Browse the repository at this point in the history
…ckerfile

Remove unnecessary step
  • Loading branch information
codeguru42 authored Sep 19, 2022
2 parents 959d135 + add0b1c commit 6ea07e7
Showing 1 changed file with 28 additions and 1 deletion.
29 changes: 28 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ on:
- '*'

jobs:
build:
api:
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand All @@ -38,3 +38,30 @@ jobs:
push: ${{ github.event_name != 'pull_request' }}
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}
nginx:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v2
- name: Docker metadata
id: meta
uses: docker/metadata-action@v3
with:
images: codeguru42/bbct-nginx
tags: |
type=semver,pattern={{version}}
type=semver,pattern={{major}}.{{minor}}
type=sha
- name: Login to DockerHub
if: github.event_name != 'pull_request'
uses: docker/login-action@v1
with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_TOKEN }}
- name: Build and push
uses: docker/build-push-action@v2
with:
context: ./nginx
push: ${{ github.event_name != 'pull_request' }}
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}

0 comments on commit 6ea07e7

Please sign in to comment.