Skip to content

update publish action to work - #66

Merged
cnotbohm merged 7 commits into
hearsaycorp:masterfrom
cnotbohm:CEL-1379
Mar 16, 2026
Merged

update publish action to work#66
cnotbohm merged 7 commits into
hearsaycorp:masterfrom
cnotbohm:CEL-1379

Conversation

@cnotbohm

Copy link
Copy Markdown
Contributor

This PR updates the publish action to build before trying to publish, which should allow us to publish properly
also updates docs to MD and makes sure that the changelog is in the pypi description

@sonarqubecloud

Copy link
Copy Markdown

@cnotbohm
cnotbohm marked this pull request as ready for review March 16, 2026 17:32
@cnotbohm
cnotbohm requested a review from a team as a code owner March 16, 2026 17:32
@cnotbohm
cnotbohm merged commit 6ce823a into hearsaycorp:master Mar 16, 2026
7 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c774dd6ec3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

name: Client Build and Publish

on:
workflow_dispatch:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Reintroduce an automatic publish trigger

This workflow now listens only to workflow_dispatch, so publishing no longer runs when a release is created (the previous client_build.yml handled release events). If your release process expects PyPI artifacts to be published automatically on release, this silently breaks that path and requires a manual run every time.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants