Skip to content

Fixing deployment action #4954

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Mar 10, 2025
Merged

Fixing deployment action #4954

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
12f03d6
start debugging the deployment action
IAlibay Mar 9, 2025
ac20c46
try splitting up the uploads
IAlibay Mar 9, 2025
21f488b
avoid the wildcards in names
IAlibay Mar 9, 2025
bcfcdec
complete workflow upgrade
IAlibay Mar 9, 2025
345cd10
try full tests for py3.13
IAlibay Mar 9, 2025
3dd3f78
remove develop trigger for deploy workflow
IAlibay Mar 9, 2025
ee99023
exclude full deploy tests with py3.13
IAlibay Mar 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 22 additions & 16 deletions .github/workflows/deploy.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
fetch-depth: 0

- name: Build wheels
uses: pypa/cibuildwheel@v2.20.0
uses: pypa/cibuildwheel@v2.23.0
with:
package-dir: package
env:
Expand All @@ -63,8 +63,9 @@ jobs:
if: |
(github.event_name == 'push' && startsWith(github.ref, 'refs/tags/package')) ||
(github.event_name == 'release' && github.event.action == 'published')
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: wheel-${{ matrix.buildplat[0] }}-${{ matrix.python }}
path: wheelhouse/*.whl
retention-days: 7

Expand All @@ -88,8 +89,9 @@ jobs:
if: |
(github.event_name == 'push' && startsWith(github.ref, 'refs/tags/package')) ||
(github.event_name == 'release' && github.event.action == 'published')
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: sdist
path: package/dist/*.tar.gz
retention-days: 7

Expand All @@ -113,8 +115,9 @@ jobs:
if: |
(github.event_name == 'push' && startsWith(github.ref, 'refs/tags/package')) ||
(github.event_name == 'release' && github.event.action == 'published')
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: sdist_tests
path: testsuite/dist/*.tar.gz
retention-days: 7

Expand All @@ -131,18 +134,18 @@ jobs:
runs-on: ubuntu-latest
needs: [build_wheels, build_sdist, build_sdist_tests]
steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: artifact
path: dist
merge-multiple: true

- name: move_test_dist
run: |
mkdir -p testsuite/dist
mv dist/mdanalysistests-* testsuite/dist

- name: upload_source_and_wheels
uses: pypa/gh-action-pypi-publish@v1.11.0
uses: pypa/gh-action-pypi-publish@v1.12.4
with:
skip_existing: true
repository_url: https://test.pypi.org/legacy/
Expand All @@ -160,18 +163,18 @@ jobs:
runs-on: ubuntu-latest
needs: [build_wheels, build_sdist, build_sdist_tests]
steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: artifact
path: dist
merge-multiple: true

- name: move_test_dist
run: |
mkdir -p testsuite/dist
mv dist/mdanalysistests-* testsuite/dist

- name: upload_tests
uses: pypa/gh-action-pypi-publish@v1.11.0
uses: pypa/gh-action-pypi-publish@v1.12.4
with:
packages_dir: testsuite/dist
skip_existing: true
Expand All @@ -190,18 +193,18 @@ jobs:
runs-on: ubuntu-latest
needs: [build_wheels, build_sdist, build_sdist_tests]
steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: artifact
path: dist
merge-multiple: true

- name: move_test_dist
run: |
mkdir -p testsuite/dist
mv dist/mdanalysistests-* testsuite/dist

- name: upload_source_and_wheels
uses: pypa/gh-action-pypi-publish@v1.11.0
uses: pypa/gh-action-pypi-publish@v1.12.4

upload_pypi_mdanalysistests:
if: |
Expand All @@ -216,18 +219,18 @@ jobs:
runs-on: ubuntu-latest
needs: [build_wheels, build_sdist, build_sdist_tests]
steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: artifact
path: dist
merge-multiple: true

- name: move_test_dist
run: |
mkdir -p testsuite/dist
mv dist/mdanalysistests-* testsuite/dist

- name: upload_tests
uses: pypa/gh-action-pypi-publish@v1.11.0
uses: pypa/gh-action-pypi-publish@v1.12.4
with:
packages_dir: testsuite/dist

Expand All @@ -249,6 +252,9 @@ jobs:
# Multiple deps don't like windows
- os: windows-latest
type: "FULL"
# Chemfiles won't install over py3.13
- python-version: "3.13"
type: "FULL"
env:
MPLBACKEND: agg

Expand Down
Loading