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

Attach namespace extensions to the root of the doc #3890

Merged
merged 7 commits into from
Jul 18, 2024

Conversation

MaggieKimani1
Copy link
Contributor

@MaggieKimani1 MaggieKimani1 commented Jul 18, 2024

This PR seeks to update the emitter by adding the extensions defined on a namespace to the root of the OpenApi document.
Fixes #3837

@microsoft-github-policy-service microsoft-github-policy-service bot added the emitter:openapi3 Issues for @typespec/openapi3 emitter label Jul 18, 2024
@MaggieKimani1 MaggieKimani1 changed the title attache extensions defined on a namespace to the root of the doc Attach namespace extensions to the root of the doc Jul 18, 2024
packages/openapi3/test/openapi-output.test.ts Outdated Show resolved Hide resolved
packages/openapi3/test/openapi-output.test.ts Outdated Show resolved Hide resolved
packages/openapi3/test/openapi-output.test.ts Outdated Show resolved Hide resolved
packages/openapi3/test/openapi-output.test.ts Outdated Show resolved Hide resolved
packages/openapi3/test/openapi-output.test.ts Outdated Show resolved Hide resolved
packages/openapi3/test/openapi-output.test.ts Outdated Show resolved Hide resolved
Co-authored-by: Timothee Guerin <timothee.guerin@outlook.com>
@timotheeguerin timotheeguerin added this pull request to the merge queue Jul 18, 2024
Merged via the queue into microsoft:main with commit 8d8e454 Jul 18, 2024
18 checks passed
@MaggieKimani1 MaggieKimani1 deleted the mk/set-root-extensions branch July 19, 2024 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
emitter:openapi3 Issues for @typespec/openapi3 emitter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update emitter to set extensions at the root of the document
2 participants