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.

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

Publish wheels #9

Merged
merged 10 commits into from
Oct 24, 2024
Merged

Publish wheels #9

merged 10 commits into from
Oct 24, 2024

Conversation

kmurphy4
Copy link
Collaborator

@kmurphy4 kmurphy4 commented Sep 26, 2024

@kmurphy4 kmurphy4 force-pushed the feature/publish-wheels branch from 4fb0b4a to 1d8f30c Compare September 26, 2024 00:30
We only use `python3.9` in production at the moment.  Soon, we'll
migrate to `python3.12`.  We likely won't ever use any of the other
ones..
@kmurphy4 kmurphy4 force-pushed the feature/publish-wheels branch from 1d8f30c to 762765b Compare September 26, 2024 00:31
@kmurphy4 kmurphy4 force-pushed the feature/publish-wheels branch 2 times, most recently from f1229cf to b322ee7 Compare October 23, 2024 06:22
You can verify that this is working correctly by running

    $ python -m build --wheel
    $ pip install dist/pypff_stubs-0.1.0-py3-none-any.whl
    $ mypy -c 'import pypff; reveal_type(pypff.file)'
    <string>:1:27: note: Revealed type is "def () -> pypff.file.file"
    Success: no issues found in 1 source file

This is based on https://mypy.readthedocs.io/en/latest/installed_packages.html#creating-pep-561-compatible-packages
and https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html.
Otherwise, pushing to a branch with an open PR will trigger two
builds.
@kmurphy4 kmurphy4 force-pushed the feature/publish-wheels branch 2 times, most recently from b826892 to d0baf4c Compare October 23, 2024 06:26
@kmurphy4 kmurphy4 marked this pull request as ready for review October 23, 2024 06:29
@kmurphy4
Copy link
Collaborator Author

cc @jordan-smith @donnelly-kyle

@ztravis
Copy link

ztravis commented Oct 23, 2024

LGTM

@kmurphy4 kmurphy4 merged commit ae1631c into master Oct 24, 2024
1 check passed
@kmurphy4 kmurphy4 deleted the feature/publish-wheels branch October 24, 2024 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants