Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ci: add verbose logging to PyPI publishing #3314

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.

Sign up for GitHub

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

Jump to bottom
Merged
mergify merged 1 commit into from
Apr 24, 2025
Merged

ci: add verbose logging to PyPI publishing #3314

mergify merged 1 commit into from
Apr 24, 2025

Conversation

@nathan-weinberg
Copy link
Member

@nathan-weinberg nathan-weinberg commented Apr 24, 2025 •
edited
Loading

we sometimes get failures with the pypi publish action

add verbose logging to make it easier to debug

see: https://github.com/pypa/gh-action-pypi-publish/tree/76f52bc884231f62b9a034ebfe128415bbaabdfc/?tab=readme-ov-file#for-debugging

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

@nathan-weinberg
ci: add verbose logging to PyPI publishing
d596d4e 
we sometimes get failures with the pypi publish action

add verbose logging to make it easier to debug

Signed-off-by: Nathan Weinberg <[email protected]>
@mergify mergify bot added the CI/CD Affects CI/CD configuration label Apr 24, 2025
@nathan-weinberg nathan-weinberg requested a review from a team April 24, 2025 17:07
@danmcp danmcp removed the request for review from a team April 24, 2025 17:14
@mergify mergify bot merged commit 67769c6 into instructlab:main Apr 24, 2025
14 checks passed
@nathan-weinberg nathan-weinberg deleted the test-pypi-v branch April 24, 2025 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ktdreyer ktdreyer ktdreyer approved these changes

+1 more reviewer

@danmcp danmcp danmcp approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

CI/CD Affects CI/CD configuration

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nathan-weinberg @danmcp @ktdreyer