Skip to content

Commit

Permalink
Merge pull request #21 from metagenlab/dev
Browse files Browse the repository at this point in the history
Fix github workflows
  • Loading branch information
farchaab authored Aug 22, 2024
2 parents a31e590 + 0626525 commit aad09fe
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/docker-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,12 @@ name: Publish Docker image
on:
push:
branches: [main]
paths:
- "Dockerfile"
- ".github/workflows/docker-publish.yml"
- "tests/**"
- "assembly_finder/**"
- "setup.py"
release:
types: [published]

Expand Down Expand Up @@ -35,9 +41,6 @@ jobs:
uses: docker/build-push-action@v3
if: github.event_name == 'push'
with:
# All available with python:3.X-slim are:
# platforms: linux/386,linux/amd64,linux/arm/v5,linux/arm/v7,linux/arm64/v8,linux/ppc64le,linux/s390x
# But 32-bit binaries likely require compilation from source so stick with linux/amd64 and linux/arm64 for now
platforms: linux/amd64,linux/arm64
push: true
tags: |
Expand All @@ -47,9 +50,6 @@ jobs:
uses: docker/build-push-action@v3
if: github.event_name == 'release'
with:
# All available with python:3.X-slim are:
# platforms: linux/386,linux/amd64,linux/arm/v5,linux/arm/v7,linux/arm64/v8,linux/ppc64le,linux/s390x
# But 32-bit binaries likely require compilation from source so stick with linux/amd64 and linux/arm64 for now
platforms: linux/amd64,linux/arm64
push: true
tags: |
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/unit-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,14 @@ on:
push:
branches: ["main"]
paths:
- ".github/workflows/unit-tests.yaml"
- ".github/workflows/unit-tests.yml"
- "tests/**"
- "assembly_finder/**"
- "setup.py"
pull_request:
branches: ["main"]
paths:
- ".github/workflows/unit-tests.yaml"
- ".github/workflows/unit-tests.yml"
- "tests/**"
- "assembly_finder/**"
- "setup.py"
Expand Down Expand Up @@ -80,7 +80,7 @@ jobs:
activate-environment: assembly_finder
python-version: ${{ matrix.python-version }}
auto-activate-base: false
- name: "Test and generate coverage report for Linux python ${{ matrix.python-version }}"
- name: "Test and generate coverage report for OSX python ${{ matrix.python-version }}"
run: |
python -m pip install --upgrade pip
python -m pip install pytest coverage
Expand Down

0 comments on commit aad09fe

Please sign in to comment.