Skip to content

Conversation

swathipil
Copy link
Member

@swathipil swathipil commented Jul 22, 2025

Note: To generate pyproject.toml, basic-setup-py will have to be removed from tspconfig.yaml.

Only the generated pyproject.toml has been added to the PR for testing purposes. All other generated code changes are outside the scope of the pyproject.toml change.

FYI: The other generated code requires manual changes as outlined here: https://gist.github.com/mccoyp/1df65df4102d158c7b75aee4b4b173cc
Due to undesired behavior described in this issue: microsoft/typespec#7715

@swathipil swathipil force-pushed the swathipil/kv/keys-pyproject-codegen branch from ddb6f77 to 4bc1d8a Compare July 22, 2025 17:54
@swathipil swathipil force-pushed the swathipil/kv/keys-pyproject-codegen branch from a4f6eb5 to c426c81 Compare August 4, 2025 19:33
Copy link
Member

@mccoyp mccoyp left a comment

Choose a reason for hiding this comment

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

Aside from the _metadata.json file location and generated model namespaces in docstrings (which both seem to be getting addressed), things look good!

@swathipil swathipil closed this Aug 6, 2025
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Azure SDK for Key Vault Aug 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants