Skip to content
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

Add initial test pypi release workflow #5052

Merged
merged 8 commits into from
Mar 26, 2025
Merged

Add initial test pypi release workflow #5052

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d34460e
Add initial test pypi release workflow
marthacryan Feb 21, 2025
0f25bd4
Syntax error
marthacryan Feb 21, 2025
efe9451
Raise node version and use verbose flag for debugging
marthacryan Feb 21, 2025
bf7874e
build
marthacryan Mar 26, 2025
aba1f3d
Only run on push to main or release branches
marthacryan Mar 26, 2025
4ffb5da
Merge branch 'main' into release-testpypi-main
marthacryan Mar 26, 2025
d0c2ec4
Add workflow dispatch for manual triggering github actions
marthacryan Mar 26, 2025
be868b4
Merge branch 'release-testpypi-main' of github.com:plotly/plotly.py i…
marthacryan Mar 26, 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
67 changes: 67 additions & 0 deletions .github/workflows/test-release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
name: Publish Python 🐍 distribution 📦 to PyPI and TestPyPI

on:
workflow_dispatch

jobs:
build:
name: Build distribution 📦
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4
with:
persist-credentials: false
- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: "3.x"

- name: Install Node
uses: actions/setup-node@v2
with:
node-version: '18'

- name: Install npm dependencies
run: |
cd js
npm ci --verbose
npm run build --verbose

- name: Install pypa/build
run: >-
python3 -m
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason why this is python3 instead of Python? Are we using the wrong container image? There should be one that comes with Python already set up and installed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this was just what was in the sample in the python packing documentation for releasing with GitHub actions

pip install
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it worth using uv here? no external dependencies here so not a lot to reconcile but nonetheless.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I don't know that it is worth it - we'd just be installing one package

build
--user
- name: Build a binary wheel and a source tarball
run: python3 -m build
- name: Store the distribution packages
uses: actions/upload-artifact@v4
with:
name: python-package-distributions
path: dist/

publish-to-testpypi:
name: Publish Python 🐍 distribution 📦 to TestPyPI
needs:
- build
runs-on: ubuntu-latest

environment:
name: testpypi
url: https://test.pypi.org/p/plotly

permissions:
id-token: write # IMPORTANT: mandatory for trusted publishing

steps:
- name: Download all the dists
uses: actions/download-artifact@v4
with:
name: python-package-distributions
path: dist/
- name: Publish distribution 📦 to TestPyPI
uses: pypa/gh-action-pypi-publish@release/v1
with:
repository-url: https://test.pypi.org/legacy/