-
Notifications
You must be signed in to change notification settings - Fork 25
Conversation
friendly ping @Waidhoferj |
@@ -3,6 +3,7 @@ name: Build and Release | |||
on: | |||
release: | |||
types: [published] | |||
pull_request: |
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.
@kafonek before trying the deploy, is there a reason that we trigger the wheel build on PRs?
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.
My goal here is to raise some of the errors we've been seeing before getting to a release action.
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.
Might be a bit overkill to run it on every PR, since we only need to check on PRs immediately before a release. Maybe we could use workflow_dispatch
instead?
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.
I'm open to using that and manually kicking off actions if you want to add me to the repo with write access 🙏 I feel bad that there's so many release action failures based on some CI PR's I've put in.
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.
I really appreciate the work you put into this. Wish it wasn't so hard to verify. Unfortunately, I don't have control over read/write access to the repo, but I think @dmonad can help with this.
@Waidhoferj do you mind deleting and re-releasing 0.5.6 to see if it will build and upload wheels to pypi this time? Or bump the version and release 0.5.7? Thanks! |
I see v0.5.6 as the latest release on GitHub, even though there is a v0.5.7. |
Fixes for the failures we saw in #115 / https://github.com/y-crdt/ypy/actions/runs/4167129650
py-dissimilar
CI https://github.com/messense/py-dissimilar/blob/main/.github/workflows/CI.yml