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

[Storage] Initial code-gen v0.4.0 (with some dependencies) #1958

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vincenttran-msft
Copy link
Member

@vincenttran-msft vincenttran-msft commented Dec 13, 2024

Purpose:
This pull-request contains the initial output of the Rust tsp-client generator. This pull request contains necessary changes to the main repo to support successful code generation including:

emitter-package.json

  • Updated "@azure-tools/typespec-rust" from v0.3.0 -> v0.4.0
  • Added missing dependency @azure-tools/typespec-azure-rulesets required for code-gen
    Note: We may want to consider having some automation to open a PR that automatically bumps these dependencies when a new version is released- I've bumped into this issues many times working between v0.2.0 and now. The above dependency version can get out of sync with the devDependencies and can be slightly difficult to diagnose if not prudently scanning versions for mismatches

TODOs to get this in a merge-able state:

1. feature/blob-tsp needs to land in a stable state that can be generated from without hand-edits
Relevant GH Issue: https://github.com/Azure/typespec-rust/issues/60
Currently we are generating against this commit, which contains hand-edits to remove the pageable operations w/ continuation tokens which are not yet supported.

2. Barring the above issue, the initial output of the generated code does not successfully compile.
Relevant GH Issue: https://github.com/Azure/typespec-rust/issues/183

Once the above dependencies are complete, we can look to upgrade this from Draft.

Note: One-off to look into- may need to add generated code to cspell.json or add the "new" words to the definitions.

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