Skip to content

Commit

Permalink
ci: Refactor workflow configuration and Makefile targets
Browse files Browse the repository at this point in the history
- Remove triggering workflow on certain pull request types
- Update Makefile to remove `install` step from `publish` target
- Update Makefile to include `install` step in `publish-package` and `publish-image` targets
  • Loading branch information
johnnyhuy committed Apr 21, 2024
1 parent c85583a commit 8a391ce
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ on:
push:
branches:
- main
pull_request:
types: [opened, reopened, synchronize]

permissions: write-all

Expand Down
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@ build:
build-image:
docker compose build

publish: install publish-package publish-image
publish: publish-package publish-image

publish-package:
publish-package: install
rm -rdf dist
$(MAKE) build
ifdef DRY_RUN
Expand All @@ -63,7 +63,7 @@ else
$(VENV) python -m twine upload dist/*
endif

publish-image: login-ghcr
publish-image: install login-ghcr
ifdef CI
docker buildx build \
-t ghcr.io/echohello-dev/transcribe-me:latest \
Expand Down

0 comments on commit 8a391ce

Please sign in to comment.