Skip to content

Commit 3f7821a

Browse files
authored
fix: publish/release workflow (#73)
Yay!
1 parent 914f13a commit 3f7821a

File tree

3 files changed

+6
-79
lines changed

3 files changed

+6
-79
lines changed

.github/workflows/publish-pypi.yml renamed to .github/workflows/build-publish-pypi.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
4040
- name: Build package using Hatch
4141
run: |
42-
hatch build
42+
hatch run build:check
4343
echo ""
4444
echo "Generated files:"
4545
ls -lh dist/
@@ -49,10 +49,13 @@ jobs:
4949
with:
5050
name: python-package-distributions
5151
path: dist/
52+
# The publish step will only run on a release from the pyopensci repository
53+
# There are several checks in place including setting up the pypi environment in
54+
# GitHub so that only people on your maintainer team can allow the deployment
5255
publish:
5356
name: >-
5457
Publish Python 🐍 distribution 📦 to PyPI
55-
if: github.repository_owner == 'pyopensci'
58+
if: github.repository_owner == 'pyopensci' && github.event_name == 'release'
5659
needs:
5760
- build
5861
runs-on: ubuntu-latest

.github/workflows/release.yml

Lines changed: 0 additions & 77 deletions
This file was deleted.

changelog.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66

77
This release fixes many issues with the original template run in "full mode"
88

9+
* Fix: release.yml broken and combine publish and release builds (@lwasser, #71)
910
* Add: fix ci, linting, docs and tests run docs separately (@lwasser)
1011
* Fix readthedocs build issues fixed by updating .readthedocs.yaml to install docs dependencies and build docs directly & redundant api docs (@lwasser)
1112
* Add: Move to optional-dependency groups and call those groups in hatch envs (@lwasser)

0 commit comments

Comments
 (0)