Skip to content

Conversation

@kevinjqliu
Copy link
Contributor

Which issue does this PR close?

  • Closes #.

What changes are included in this PR?

Follow up to #1836
Also add verbose: true to pypa/gh-action-pypi-publish in .github/workflows/release_python.yml

Are these changes tested?

@mbutrovich
Copy link
Collaborator

Thanks @kevinjqliu! In the event of failure this gives output like what you posted in this comment?

#1836 (comment)

Just trying to get a bit more context.

@kevinjqliu
Copy link
Contributor Author

yes! The INFO is what is provided by setting verbose to True. otherwise, we just get the error

ERROR    HTTPError: 400 Bad Request from https://test.pypi.org/legacy/          
         Bad Request                                                            

which isnt helpful

Copy link
Collaborator

@mbutrovich mbutrovich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes! The INFO is what is provided by setting verbose to True. otherwise, we just get the error

ERROR    HTTPError: 400 Bad Request from https://test.pypi.org/legacy/          
         Bad Request                                                            

which isnt helpful

Makes sense, thank you!

Copy link
Contributor

@liurenjie1024 liurenjie1024 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kevinjqliu !

@liurenjie1024 liurenjie1024 merged commit 0107f17 into apache:main Nov 13, 2025
18 checks passed
@kevinjqliu kevinjqliu deleted the kevinjqliu/release_python-verbose branch November 13, 2025 18:19
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.

3 participants