-
Notifications
You must be signed in to change notification settings - Fork 101
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
Use pyproject toml and enable workflow for publishing on PyPI #480
Use pyproject toml and enable workflow for publishing on PyPI #480
Conversation
92e8d38
to
81bf1b5
Compare
81bf1b5
to
7e29630
Compare
c26888d
to
257b46a
Compare
Some errors from building python-podman might go away if we rebase onto #476 as typing_extensions.py is going away. So I am trying to rebase onto these commits now although the only important ones are the last two at the moment |
dcf1909
to
309c0ca
Compare
@inknos still working on this? |
yes, waiting on #476 to be merged first |
309c0ca
to
5603867
Compare
5603867
to
ca51d7a
Compare
Fix errors for tyope annotation for list, dict, type and tuple Example: UP006: Use `list` instead of `List` for type annotation Signed-off-by: Nicola Sella <[email protected]>
Signed-off-by: Nicola Sella <[email protected]>
4e5da81
to
6d66eaa
Compare
6380998
to
6d66eaa
Compare
I am investigating why the package |
6d66eaa
to
0ba67f9
Compare
LGTM |
a bit of tweaking is still required as builds for centos9 and epel9 are currently failing, but I identified the issue |
Signed-off-by: Nicola Sella <[email protected]>
Signed-off-by: Nicola Sella <[email protected]>
0ba67f9
to
945693b
Compare
@jwhonce I think we are good after the tests pass |
LGTM @lsm5 PTAL Thanks |
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.
Don't understand this much TBH, and IIUC we'll see the results of GHA only after a new release is created .
Everything's green so LGTM.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: inknos, jwhonce, lsm5 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
This PR incorporates all the building tools in pyproject.toml while keeping most of the changes in the
It also proposes to auto-build pushes and publish signed artifacts on PyPI testing. The process also automates tagged commits publishing to PyPI.
Publishing on PyPI follows this guide
Requires: #476