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

feat(ts): adds connect-es generated code (#184) #157

Closed
wants to merge 1 commit into from
Closed

Conversation

ygrishajev
Copy link
Contributor

@ygrishajev ygrishajev commented Apr 24, 2024

This PR introduces necessary updates to support connect-es in addition to ts-proto generated code which is needed for our indexer application. It includes the generation of provider code and updates to the package exports to include connect-es modules. Additionally, the directory structure has been updated along with the dynamic exports generator script to reflect these changes.

Key Changes

  • Provider Code Generation: Added necessary configurations to generate provider code using connect-es.
  • Package Exports: Updated to include connect-es exports, ensuring that all generated modules are correctly exposed for use in the application.
  • Directory Structure: Revised the project's directory structure to accommodate the new code generation strategy and organization.
  • Scripts Update:
    • make/codegen.mk: Updated commands for connect-es generation alongside existing ts-proto generation.
    • script/protocgen-legacy.sh: Modified to support the new generation requirements.
    • ts/script/generate-exports.js: Enhanced to dynamically generate exports based on the updated directory structure and generation strategies.

Implementation Details

The majority of changes in this PR involve file movements to better organize the generated code. The scripts mentioned above have been updated to facilitate these changes and to automate the generation and export of modules.

Notes

Most changes in this PR are related to the restructuring of files and directories; the most relevant code changes are found in the scripts listed above. It is recommended to review these scripts for detailed understanding of the new generation and export processes.

@Redm4x
Copy link

Redm4x commented Apr 24, 2024

Would be nice to have examples of how to use both grpc clients in the ts README

@ygrishajev
Copy link
Contributor Author

Would be nice to have examples of how to use both grpc clients in the ts README

Ok, but I’ll attempt to avoid this pr first as discussed

@ygrishajev ygrishajev closed this May 2, 2024
@ygrishajev ygrishajev deleted the feat/ts branch May 2, 2024 08:26
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.

2 participants