Skip to content

feat: add support for building client from scripts #97

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

NishantJoshi00
Copy link
Member

@NishantJoshi00 NishantJoshi00 commented Jul 29, 2025

This pull request introduces a new workflow for automatically regenerating SDK clients when proto files change, updates the build system to reflect these changes, and removes outdated refactor documentation. The most important changes include adding a GitHub Actions workflow for proto code generation, modifying the Rust workspace configuration, updating the Makefile to include a target for regenerating SDKs, and removing obsolete refactor documentation.

Automation for Proto Code Generation:

  • .github/workflows/proto-codegen.yml: Added a GitHub Actions workflow to detect changes in proto files, regenerate SDK clients, and create a pull request with the updated clients.

Build System Updates:

  • Cargo.toml: Updated the Rust workspace configuration to move sdk/rust-grpc-client to the exclude list and added tools/proto-codegen to the workspace members.

Makefile Enhancements:

  • Makefile: Added a regenerate-sdks target to regenerate SDK clients from proto files and updated the help target to include this new command. [1] [2] [3]

Documentation Cleanup:

  • Removed outdated refactor documentation files, including refactor-docs/00-quick-reference.md, refactor-docs/01-refactor-overview.md, and refactor-docs/02-phase1-infrastructure.md, as they are no longer relevant to the current workflow. [1] [2] [3]

@NishantJoshi00 NishantJoshi00 requested a review from a team as a code owner July 29, 2025 09:16
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.

1 participant