-
Notifications
You must be signed in to change notification settings - Fork 75
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
Fix enum values to PascalCase and replace return type of a method returning a stream from RedableStream to Response #12
Merged
Conversation
This file contains 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
…e instead of ReadableStream for the streaming scenario.
🤖 AutoRest automatic publish job 🤖success (version: 2.0.67) |
|
qiaozha
added a commit
that referenced
this pull request
Nov 4, 2023
* inital-commit-of-hierarchy-support * add model namespace name * reserve work * revert change in openai modular as of openai generic * generate operations * add classical operation hierarchy layers and reserve work * organize classic operation as hierarchy layered * change typespec to have simpler names * resolve operation conflicts with full namespace path * add namespace prefix for model only when conflicts * add edge case for model and operation name conflicts and resolve model conflicts if detect * enableModelNamespace as true for anomaly detector * fix hierarchy layer info bug * do not rename in Modular but rename in RLC if hierarchy client and conflict exists * fix model conflicts detect condition * update edge case generation * update package.json with hierarchy subpath * manual change for export the operation interface * reserve work * generate options in a centralized place * update generated code for edge case * loadtesting-with-hierarchy-false (#9) * loadtesting-with-hierarchy-false * do not return namespace hierarchies * use interface name when enable operation group in rlc * fix package.json * update generic generated files with just pure import order change (#10) * loadtesting-with-hierarchy-false * do not return namespace hierarchies * use interface name when enable operation group in rlc * fix package.json * update generic generated files just import order chaneg * disable-hierarchy-client-and-not-introduce-extra-changes (#11) * disable-hierarchy-client-and-not-introduce-extra-changes * fix model import issues * reserve work * reserve work * fix hierarchy layer overload * fix generation failure * fall back to enableOperationGroup logic if hierarchyClient is false * fix package.json and api layer export * fix smoke-test * fix lint error * tryfix ci * add operation group name back if hierarchy client is false * use operation group name if hierarchy client is false * rename operation if enableOperationGroup true and hierarchyClient false * do not include operation group name duplicated * only return interface name when enableOperationGroup true and hierarchyClient false * disable hierarchyClient for chatApi * Update packages/typespec-test/test/chatApi_modular/generated/typespec-ts/package.json * Update packages/typespec-test/test/loadtesting_modular/tspconfig.yaml * enableOperationGroup for service with operationGroup decorator * to not get namespace hierarchy if hierarchyClient is false (#12) * hierarchy-namespace-in-client-tsp * fix client structure with operations * update smoke-test * undo enable operation group for clients
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.
No description provided.