fix: run pypi / docker upload from published draft releases (#2461)
Draft releases don't trigger the workflows (that's good!) but since they only Commit history before merge: * fix: run pypi upload from published draft releases * Fix broken task list markup in PR template * change docker workflow to build on release publish Co-authored-by: Richard Si <63936253+ichard26@users.noreply.github.com>
This commit is contained in:
parent
a8b4665e7d
commit
79575f3376
6
.github/PULL_REQUEST_TEMPLATE.md
vendored
6
.github/PULL_REQUEST_TEMPLATE.md
vendored
@ -18,9 +18,9 @@
|
|||||||
Tests are required for bugfixes and new features. Documentation changes
|
Tests are required for bugfixes and new features. Documentation changes
|
||||||
are necessary for formatting and most enhancement changes. -->
|
are necessary for formatting and most enhancement changes. -->
|
||||||
|
|
||||||
- [] Add a CHANGELOG entry if necessary?
|
- [ ] Add a CHANGELOG entry if necessary?
|
||||||
- [] Add / update tests if necessary?
|
- [ ] Add / update tests if necessary?
|
||||||
- [] Add new / update outdated documentation?
|
- [ ] Add new / update outdated documentation?
|
||||||
|
|
||||||
<!-- Just as a reminder, everyone in all psf/black spaces including PRs
|
<!-- Just as a reminder, everyone in all psf/black spaces including PRs
|
||||||
must follow the PSF Code of Conduct (link below).
|
must follow the PSF Code of Conduct (link below).
|
||||||
|
2
.github/workflows/docker.yml
vendored
2
.github/workflows/docker.yml
vendored
@ -5,7 +5,7 @@ on:
|
|||||||
branches:
|
branches:
|
||||||
- "main"
|
- "main"
|
||||||
release:
|
release:
|
||||||
types: created
|
types: [published]
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
docker:
|
docker:
|
||||||
|
2
.github/workflows/pypi_upload.yml
vendored
2
.github/workflows/pypi_upload.yml
vendored
@ -2,7 +2,7 @@ name: pypi_upload
|
|||||||
|
|
||||||
on:
|
on:
|
||||||
release:
|
release:
|
||||||
types: created
|
types: [published]
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
build:
|
build:
|
||||||
|
Loading…
Reference in New Issue
Block a user