-
Notifications
You must be signed in to change notification settings - Fork 8
ci: use pyproject.toml Python version for build wheel, deploy docs, release #203
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.
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
Conversation
bobleesj
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sbillinge ready for review. just like test-on-pr, user can also mnually specifiy the python version in the .yml file
| default: 'PROJECT_NAME' | ||
| required: true | ||
| type: string | ||
| python_version: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this .yml is for non-pure Python - use the pyython-version from the main _build-wheel-..yml file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bobleesj do we need a comment in the code with this information so we don't forget in the future? Not sure, just asking.
| maintainer_github_username: ${{ inputs.maintainer_github_username }} | ||
|
|
||
| build-wheel: | ||
| get-python-version: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this .yml is for the main entry point for when git push upstream <tag> is called.
like in test-on-pr, we grab the latest Python version.
| default: false | ||
| required: true | ||
| type: boolean | ||
| python_version: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for builidng wheel, use the Python version passed from the top level
| description: 'Python version for Conda environment' | ||
| default: 3.14 | ||
| required: false | ||
| description: 'Python version to use for building documentation' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for building docs, again, same logic.
| uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: '3.14' | ||
| python-version: ${{ inputs.python_version }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for uploading to PyPI, again same logic. use the Python version passed from the main build-wheel-upload .yml file
|
@sbillinge ready for review |
Merge pull request #203 from bobleesj/build-get-pt-version v0
Fixes scikit-package/scikit-package#632
Just like on test-on-pr, this PR grabs the latest Python version from .pyproject.toml