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

[github] Add a manual workflow to publish openai-deno-build #516

Merged
merged 2 commits into from
Nov 19, 2023

Conversation

athyuttamre
Copy link
Collaborator

@athyuttamre athyuttamre commented Nov 19, 2023

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

Adds a new GitHub workflow that can be run manually to publish the latest tag to the openai-deno-build repo. All pushes to that repo are published to https://deno.land/x/openai.

The publishing to the Deno repo will happen using the credentials of the newly set up "OpenAI Open Source" GitHub App. I've added the secrets APP_ID and APP_PRIVATE_KEY to the publish environment.

I've tested this works by:

  • Forking the openai-node and openai-deno-build repos
  • Creating my own GitHub App that is installed in both
  • Running the workflow, and confirming that the latest tag is pushed from the source repo to the destination repo

If this manual publishing step works correctly, I will update the create-releases.yml workflow to use the same steps to enable auto-publishing. To ensure we don't publish broken tags to https://deno.land/x/openai, I will disable webhooks on the openai-deno-build repo until I'm sure the publishing from openai-node to openai-deno-build works correctly.

Additional context & links

Copy link
Collaborator

@rattrayalex rattrayalex left a comment

Choose a reason for hiding this comment

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

Great!

@athyuttamre athyuttamre merged commit 6e2df3d into master Nov 19, 2023
1 check passed
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