Skip to content

Conversation

@tmathern
Copy link
Collaborator

@tmathern tmathern commented May 7, 2025

Replaces #94.

Details about the update are logged in the comments on the PR (see below).

Example runs:

@tmathern tmathern self-assigned this May 7, 2025
@tmathern tmathern changed the base branch from gpeacock/py_publish to main May 7, 2025 23:35
@tmathern tmathern changed the title ci: Update setup ci: Update setup for publishing May 7, 2025
@tmathern tmathern changed the base branch from main to gpeacock/py_publish May 7, 2025 23:51
@@ -0,0 +1,98 @@
name: Upload Test for Pypi.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Note: Came in when I merged in latest main, did not make changes here in that PR.

python3 ./tests/test_api.py
python3 ./tests/test_api.py

publish: release
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Came in with latest main too

@tmathern tmathern requested a review from gpeacock May 7, 2025 23:58
@tmathern tmathern marked this pull request as ready for review May 7, 2025 23:58
@tmathern tmathern changed the base branch from gpeacock/py_publish to main May 8, 2025 20:02
Copy link
Contributor

@gpeacock gpeacock left a comment

Choose a reason for hiding this comment

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

Thank you so much for this. It is LONG overdue!

path: dist
merge-multiple: true
- name: List contents of dist directory
run: ls -la dist/
Copy link
Contributor

Choose a reason for hiding this comment

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

probably don't need this now that it works, but I think I'd leave it in anyway, just to be sure

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sounds good. We can remove it in subsequent PRs. And you're right, it's nice to have here to see what goes in.

@tmathern tmathern merged commit b41be67 into main May 9, 2025
42 of 44 checks passed
@tmathern tmathern deleted the mathern/py-publish-tmn branch May 9, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants