-
Notifications
You must be signed in to change notification settings - Fork 30
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
Release to both github and pypi via workflow #188
Conversation
Blocked by pypi/support#1735 |
45d685b
to
be6d956
Compare
With poetry 1.2, we don't need to corrupt the urlmatch checksum anymore |
Hey @ahmubashshir I have updated the workflow a bit. But github isn't allowing me to push to that from the CLI for some reason (permission denied). Could you please try adding me as a collaborator to your fork so I can commit there? |
@ahmubashshir Seems like there are conflicts. Can be fixed with I can't do a force-push on your branch, so you'll need to run that. Then we should be able to merge this |
On October 23, 2022 7:18:32 AM UTC, Krut Patel ***@***.***> wrote:
***@***.*** Seems like there are conflicts. Can be fixed with `git rebase master --strategy-option=theirs`.
I can't do a force-push on your branch, so you'll need to run that. Then we should be able to merge this
My box is borked. That'll take a few more days...
|
Closes iamkroot#181 Signed-off-by: Mubashshir <[email protected]>
Signed-off-by: Mubashshir <[email protected]>
* Update action versions * Use cache directly from setup-python action * Specify python-version matrix for test job * Remove pytest install step
3453c65
to
0e178b1
Compare
does it work now? |
Looks good! |
Changes:
Requirements:
Pypi.org auth token added as a secret named
PYPI_TOKEN