ci(docs): dispatch convoy-go, convoy.rb and convoy-php SDK generation - #2740
Merged
Conversation
adds the three repos to the trigger-sdk-repos matrix. each repo owns its generator behind the shared sdk_generation.yaml filename and inputs (convoy-go: oapi-codegen; convoy.rb / convoy-php: OpenAPI Generator). SDK_REPOS_PAT must cover the new repos.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
convoy-go,convoy.rb, andconvoy-phpto the SDK regeneration dispatch matrix. Each repo owns its generator behind the sharedsdk_generation.yamlfilename and inputs (convoy-go: oapi-codegen; convoy.rb / convoy-php: OpenAPI Generator), so the dispatcher contract is unchanged.SDK_REPOS_PATmust be able to dispatch workflows and open PRs on all six SDK repos.Companion PRs adding the pipelines: frain-dev/convoy-go#60, frain-dev/convoy.rb#20, frain-dev/convoy-php#11.
Test plan
SDK_REPOS_PATcovers the new repos, a dispatcher run should fan out to all six repos (no-diff repos skip PR creation).Note
Low Risk
Docs and CI matrix only; no runtime code. Operational risk is ensuring
SDK_REPOS_PATpermissions include the three new repos before the next dispatch.Overview
Extends the OpenAPI → SDK regeneration dispatcher so
speakeasy-sdk.ymlfans out to six language repos instead of three, addingconvoy-go,convoy.rb, andconvoy-phpto thetrigger-sdk-reposmatrix. Each new repo is expected to implement the samesdk_generation.yamlcontract (force,feature_branch); generators are documented as oapi-codegen for Go and OpenAPI Generator for Ruby/PHP.SPEAKEASY_SDK.mdis updated to match: generator/scope tables, pipeline diagram, and secret guidance now state thatSDK_REPOS_PAT/SDK_BOT_PATmust cover all matrix repos. The note deferring Go/Ruby/PHP codegen is removed.Reviewed by Cursor Bugbot for commit 12d886c. Bugbot is set up for automated code reviews on this repo. Configure here.