Skip to content

Commit

Permalink
Merge pull request #1380 from Sage-Bionetworks/develop-fix-workflow
Browse files Browse the repository at this point in the history
[bug fix] Updated workflow to install dependencies and root project together
  • Loading branch information
linglp authored Feb 23, 2024
2 parents 103696a + 89b7bb9 commit f802ce6
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 31 deletions.
12 changes: 3 additions & 9 deletions .github/workflows/api_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,17 +54,11 @@ jobs:
key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}

#----------------------------------------------
# install dependencies if cache does not exist
# install dependencies and root project
#----------------------------------------------
- name: Install dependencies
- name: Install dependencies and root project
if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
run: poetry install --no-interaction --no-root --all-extras

#----------------------------------------------
# install your root project, if required
#----------------------------------------------
- name: Install library
run: poetry install --no-interaction
run: poetry install --no-interaction --all-extras

#----------------------------------------------
# run API test suite
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/pdoc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,11 +65,11 @@ jobs:
key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}

#----------------------------------------------
# install dependencies if cache does not exist
# install dependencies and root project
#----------------------------------------------
- name: Install dependencies
- name: Install dependencies and root project
if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
run: poetry install --no-interaction --no-root
run: poetry install --no-interaction --all-extras

# create documentation
- run: poetry add pdoc@13.0.0
Expand Down
14 changes: 4 additions & 10 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,18 +44,12 @@ jobs:
key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}

#----------------------------------------------
# install your root project
#----------------------------------------------
- name: Install library
run: poetry install --only-root

# install dependencies and root project
#----------------------------------------------
# install dependencies if cache does not exist
#----------------------------------------------
- name: Install dependencies
- name: Install dependencies and root project
if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
run: poetry install --no-interaction --all-extras --no-root

run: poetry install --no-interaction --all-extras
#----------------------------------------------
# get current pushed tag
#----------------------------------------------
Expand Down
12 changes: 3 additions & 9 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,17 +66,11 @@ jobs:
key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}

#----------------------------------------------
# install your root project
#----------------------------------------------
- name: Install library
run: poetry install --only-root

#----------------------------------------------
# install dependencies if cache does not exist
# install dependencies and root project
#----------------------------------------------
- name: Install dependencies
- name: Install dependencies and root project
if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
run: poetry install --no-interaction --all-extras --no-root
run: poetry install --no-interaction --all-extras

#----------------------------------------------
# perform linting
Expand Down

0 comments on commit f802ce6

Please sign in to comment.