Skip to content

Commit

Permalink
Bumping up github actions
Browse files Browse the repository at this point in the history
- deprecated upload-artifacts -> upgrade to v4
- general bumping of the rest
- keep output of failed tests
- megalinter v8 new report structure
- Mambaforge -> Miniforge3
  • Loading branch information
DrYak committed Oct 3, 2024
1 parent e2ef699 commit 42e417b
Show file tree
Hide file tree
Showing 6 changed files with 49 additions and 33 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/deploy-docker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:

- name: Build and push
id: docker_build
uses: docker/build-push-action@v5
uses: docker/build-push-action@v6
with:
push: ${{ contains(github.ref, 'master') || startsWith(github.ref, 'refs/tags') }}
tags: ${{ steps.meta.outputs.tags }}
Expand Down
9 changes: 4 additions & 5 deletions .github/workflows/mega-linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
steps:
# Git Checkout
- name: Checkout Code
uses: actions/checkout@v2
uses: actions/checkout@v4
with:
token: ${{ secrets.PAT || secrets.GITHUB_TOKEN }}
fetch-depth: 0
Expand All @@ -37,7 +37,7 @@ jobs:
id: ml
# You can override Mega-Linter flavor used to have faster performances
# More info at https://megalinter.github.io/flavors/
uses: megalinter/megalinter@v6
uses: megalinter/megalinter@v8
env:
# All available variables are described in documentation
# https://megalinter.github.io/configuration/
Expand All @@ -48,12 +48,11 @@ jobs:
# Upload Mega-Linter artifacts
- name: Archive production artifacts
if: ${{ success() }} || ${{ failure() }}
uses: actions/upload-artifact@v2
uses: actions/upload-artifact@v4
with:
name: Mega-Linter reports
path: |
report
mega-linter.log
megalinter-reports
# Create pull request if applicable (for now works only on PR from same repository, not from forks)
- name: Create Pull Request with applied fixes
Expand Down
19 changes: 11 additions & 8 deletions .github/workflows/run_regression_tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,15 @@ jobs:

steps:
- name: Checkout repository
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
lfs: false

- name: Install conda environment dependencies
uses: conda-incubator/setup-miniconda@v2
uses: conda-incubator/setup-miniconda@v3
with:
miniforge-version: latest
miniforge-variant: Mambaforge
miniforge-variant: Miniforge3
python-version: "3.11"
mamba-version: "*"
channels: conda-forge,bioconda #,defaults # see: https://github.com/conda-forge/status/issues/144
Expand Down Expand Up @@ -75,16 +75,16 @@ jobs:
shell: bash -l {0}

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
lfs: false
# NOTE to save on bandwidth costs, GitHub Action will not use GitHub's own LFS

- name: Install conda environment dependencies
uses: conda-incubator/setup-miniconda@v2
uses: conda-incubator/setup-miniconda@v3
with:
miniforge-version: latest
miniforge-variant: Mambaforge
miniforge-variant: Miniforge3
python-version: ${{ matrix.python-version }}
mamba-version: "*"
channels: conda-forge,bioconda #,defaults # see: https://github.com/conda-forge/status/issues/144
Expand All @@ -104,9 +104,12 @@ jobs:
run: ./regression_tests.sh ${{ matrix.virus }}

- name: Archive code coverage results
uses: actions/upload-artifact@v3
if: ${{ success() }} || ${{ failure() }}
uses: actions/upload-artifact@v4
with:
name: regression test output and diffs
name: regression_test_${{ matrix.os }}_${{ matrix.python-version }}_${{ matrix.virus }}
path: |
/tmp/v-pipe_tests/*
!/tmp/v-pipe_tests/.snakemake/conda/*/
include-hidden-files: true
if-no-files-found: ignore
12 changes: 7 additions & 5 deletions .github/workflows/snakedeploy_tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:

steps:
- name: Checkout test data
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
sparse-checkout: |
tests
Expand All @@ -37,10 +37,10 @@ jobs:
# NOTE to save on bandwidth costs, GitHub Action will not use GitHub's own LFS

- name: Install conda environment dependencies
uses: conda-incubator/setup-miniconda@v2
uses: conda-incubator/setup-miniconda@v3
with:
miniforge-version: latest
miniforge-variant: Mambaforge
miniforge-variant: Miniforge3
python-version: "3.11" # temporary fix for snakemake issue #2480
mamba-version: "*"
channels: conda-forge,bioconda
Expand Down Expand Up @@ -81,11 +81,13 @@ jobs:
--keep-going
- name: Archive test results
uses: actions/upload-artifact@v3
if: ${{ success() }} || ${{ failure() }}
uses: actions/upload-artifact@v4
with:
name: test_output
name: test_output_${{ matrix.virus }}
path: |
./*
!./.git
!./.snakemake/conda/*/
include-hidden-files: true
if-no-files-found: ignore
4 changes: 2 additions & 2 deletions .github/workflows/test-docker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,15 +42,15 @@ jobs:

- name: Build and save
id: docker_build
uses: docker/build-push-action@v5
uses: docker/build-push-action@v6
with:
push: false
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}
outputs: type=docker,dest=./vpipe-docker.tar.zst,compression=zstd,compression-level=6

- name: Upload artifact
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: "( !contains(github.ref, 'master') && !startsWith(github.ref, 'refs/tags') )"
with:
name: vpipe-docker
Expand Down
36 changes: 24 additions & 12 deletions .github/workflows/tutorials.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
run: rm -rf /opt/hostedtoolcache

- name: Checkout repository
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
sparse-checkout: |
docs
Expand All @@ -33,10 +33,10 @@ jobs:
lfs: false

- name: Install conda environment dependencies
uses: conda-incubator/setup-miniconda@v2
uses: conda-incubator/setup-miniconda@v3
with:
miniforge-version: latest
miniforge-variant: Mambaforge
miniforge-variant: Miniforge3
mamba-version: "*"
channels: conda-forge,bioconda
channel-priority: strict
Expand All @@ -57,16 +57,16 @@ jobs:
run: tar --zstd -cvf vpipeinstallation.tar.zst ./docs/vp-analysis

- name: Keep installation
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: VPipeInstallation
path: vpipeinstallation.tar.zst
if-no-files-found: error

- name: Save notebooks
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: JupyterNotebooks
name: JupyterNotebooks-Install
path: ./docs/*.ipynb


Expand All @@ -93,7 +93,7 @@ jobs:
run: rm -rf /opt/hostedtoolcache

- name: Checkout repository
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
sparse-checkout: |
docs
Expand All @@ -102,18 +102,18 @@ jobs:
lfs: false

- name: Reuse installation
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: VPipeInstallation

- name: Extract V-pipe installation archive
run: tar --zstd -xvf vpipeinstallation.tar.zst

- name: Install conda environment dependencies
uses: conda-incubator/setup-miniconda@v2
uses: conda-incubator/setup-miniconda@v3
with:
miniforge-version: latest
miniforge-variant: Mambaforge
miniforge-variant: Miniforge3
python-version: "3.11"
mamba-version: "*"
channels: conda-forge,bioconda
Expand All @@ -130,10 +130,22 @@ jobs:
working-directory: ./docs
run: ./convert.sh --branch tutorial*_${{ matrix.virus }}.md

- name: Save output
if: ${{ failure() }}
uses: actions/upload-artifact@v4
with:
name: Tutorial_${{ matrix.virus }}
path: |
./docs/*.ipynb
./docs/vp-analysis/work*
!./docs/vp-analysis/work*/.snakemake/conda/*/
include-hidden-files: true
if-no-files-found: ignore

- name: Save notebooks
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: JupyterNotebooks
name: JupyterNotebooks-${{ matrix.virus }}
path: ./docs/*.ipynb

# - name: Publish
Expand Down

0 comments on commit 42e417b

Please sign in to comment.