Skip to content

Commit

Permalink
fix: Correct dockerfile and hail mary on linting failure
Browse files Browse the repository at this point in the history
  • Loading branch information
bryantbiggs committed Jan 22, 2025
1 parent eabbcf7 commit d14ad80
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 5 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/test-and-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,14 @@ jobs:
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 #v4.2.2
with:
fetch-depth: 0

- name: Setup build environment
uses: ./.github/actions/setup-build

- name: Unit test
run: |
PATH=$PATH:$(go env GOPATH)/bin make build
PATH=$PATH:$(go env GOPATH)/bin make unit-test-no-generate
make build
make unit-test-no-generate
lint:
name: Lint
runs-on: ubuntu-latest
Expand All @@ -27,11 +29,12 @@ jobs:
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 #v4.2.2
with:
fetch-depth: 0

- name: Setup build environment
uses: ./.github/actions/setup-build

- name: Lint
run: |
PATH=$PATH:$(go env GOPATH)/bin make lint
run: make lint

image:
name: Build and check image
Expand All @@ -41,6 +44,7 @@ jobs:
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 #v4.2.2
with:
fetch-depth: 0

- name: Build image
id: push
uses: docker/build-push-action@b32b51a8eda65d6793cd0494a773d4f6bcef32dc #v6.11.0
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# syntax=docker/dockerfile:1
FROM public.ecr.aws/docker/library/golang:1.22.10 AS builder
FROM public.ecr.aws/docker/library/golang:1.21.10 AS builder

WORKDIR /src
COPY . .
Expand Down

0 comments on commit d14ad80

Please sign in to comment.