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

Forks Github Actions fail on PyPi upload attempt #12

Open
filfreire opened this issue Aug 1, 2020 · 3 comments
Open

Forks Github Actions fail on PyPi upload attempt #12

filfreire opened this issue Aug 1, 2020 · 3 comments

Comments

@filfreire
Copy link
Contributor

As of commit 3e68ba2, other users' forks will run Github Actions and their pipelines will fail on the Build and Publish step (due to missing credentials).

Example (link):

Uploading distributions to https://upload.pypi.org/legacy/
Uploading questions_three-3.12.1.0-py3-none-any.whl

  0%|          | 0.00/85.3k [00:00<?, ?B/s]
  9%|▉         | 8.00k/85.3k [00:00<00:01, 52.3kB/s]
100%|██████████| 85.3k/85.3k [00:00<00:00, 272kB/s] 
NOTE: Try --verbose to see response content.
HTTPError: 403 Forbidden from https://upload.pypi.org/legacy/
Invalid or non-existent authentication information. See https://pypi.org/help/#invalid-auth for more information.
##[error]Process completed with exit code 1.

@mikeduskis @ToddBradley any advice on how people with forks can handle this? Or is it something that needs to be edited on .github/workflows/publish.yml?

From what I could find online, there are ways of adding conditions to workflow files, or each Fork "owner" would need to disable actions.

@ToddBradley
Copy link
Contributor

Hmm, interesting. I was surprised to hear the workflow is running in your fork, because GitHub's documentation says "Workflows in forked repositories don't run by default."

https://docs.github.com/en/actions/getting-started-with-github-actions/about-github-actions

I guess that part of the doc is wrong. Given that, I think the best workaround is for forkers to disable workflows altogether or modify them to their own purposes, just as they would modify any other part of the code for their purposes. Does that seem reasonable?

As an aside, do you have modifications you'd like to share with other users?

@filfreire
Copy link
Contributor Author

filfreire commented Aug 3, 2020

Given that, I think the best workaround is for forkers to disable workflows altogether or modify them to their own purposes, just as they would modify any other part of the code for their purposes. Does that seem reasonable?

Yes seems reasonable!

As an aside, do you have modifications you'd like to share with other users?

There's 3 modifications that I actually think could be useful:

  • adding a mention to the README that users should disable Github Actions under some conditions
  • a Colorized version of the Event logger that can be enabled/disabled via execution flag.
  • a document explaining how to create a "plugin" for questions-three - for example - let's say I want to create a simple HTML reporter, or a text report that also shares curl format of requests sent on the HTTPClient, or even the colorized event logger mentioned above: how could a contributor go about of either creating a separate "plugin" repo or submitting a PR to be added to the main questions-three codebase, depending on the feature, with an easy-to-follow example.

@ToddBradley
Copy link
Contributor

All three of those sound useful to me. Wanna put together a PR for each one?

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

No branches or pull requests

2 participants