Skip to content

Commit 0f6cb45

Browse files
committed
change publish.yml trigger 'release'; remove old publish_package.yml
1 parent eb4ff2d commit 0f6cb45

File tree

3 files changed

+3
-42
lines changed

3 files changed

+3
-42
lines changed

.github/workflows/publish.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
name: "Publish verda to PyPI"
22

3-
# TODO(shamrin) switch to tags or releases
43
on:
5-
pull_request:
4+
release:
5+
types: [published]
66

77
jobs:
88
deploy:
@@ -39,4 +39,3 @@ jobs:
3939
# we use Trusted publishing, no keys necessary: https://docs.astral.sh/uv/guides/integration/github/#publishing-to-pypi
4040
- name: Publish
4141
run: uv publish
42-

.github/workflows/publish_package.yml

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

tests/unit_tests/test_datacrunch.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ def test_datacrunch_module_deprecation():
4444

4545

4646
def test_datacrunch_constants_module():
47-
# Test that old re-exports in datacrunch.datacrunch module still work, but warn
47+
# Test that old re-exports in datacrunch.datacrunch (sub)module still work, but warn
4848

4949
with pytest.warns(DeprecationWarning, match='datacrunch.datacrunch is deprecated'):
5050
from verda.datacrunch import Constants

0 commit comments

Comments
 (0)