Skip to content

Commit de3888a

Browse files
committed
improve github actions
1 parent 700fe80 commit de3888a

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

.github/workflows/github-actions.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ env:
2424

2525
jobs:
2626
linting:
27+
# Run pre-commit (https://pre-commit.com/)
28+
# which runs pre-configured linter & autoformatter
2729
runs-on: ubuntu-latest
2830
steps:
2931
- uses: actions/checkout@v2
@@ -108,7 +110,8 @@ jobs:
108110
109111
build-n-publish-dummy:
110112
runs-on: ubuntu-latest
111-
needs: [linting, test]
113+
needs: [linting, test, build_doc_dryrun]
114+
if: github.event_name != 'pull_request'
112115
steps:
113116
- uses: actions/checkout@master
114117
- name: Set up Python 3.7
@@ -123,15 +126,14 @@ jobs:
123126
# # Although working and recommended, test-pypi has a limit
124127
# # in the size of projects so it's better to avoid publishing
125128
# # until there is a way to garbage collect these dummy releases
126-
# if: github.event_name != 'pull_request'
127129
# uses: pypa/gh-action-pypi-publish@master
128130
# with:
129131
# password: ${{ secrets.test_pypi_token }}
130132
# repository_url: https://test.pypi.org/legacy/
131133

132134
build-n-publish:
133135
runs-on: ubuntu-latest
134-
needs: [linting, test]
136+
needs: [linting, test, build_doc_dryrun, build-n-publish-dummy]
135137
if: github.event_name == 'create' && startsWith(github.ref, 'refs/tags/v')
136138
steps:
137139
- uses: actions/checkout@master

0 commit comments

Comments
 (0)