-
Notifications
You must be signed in to change notification settings - Fork 204
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
Adds public function for setting info object directly #3626
Adds public function for setting info object directly #3626
Conversation
…sunday/typespec into irvine/set-info-extension
…sunday/typespec into irvine/set-info-extension
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think setNamrspaceextension is the correct name setExyensiom already able to be applied to a namespace. The problem here is that we don't have a type that map to info.
@timotheeguerin your committed suggested change has broken build: export function setInfo(
program: Program,
entity: Namespace,
data: AdditionalInfo & Record<ExtensionKey, unknown>
) { } |
@timotheeguerin thanks a bunch for your input here 🙏🏾 |
Fixes #3446
By adding support for extending namespace, we will be able to add extensions to the OpenAPI
info
object via the new function -setNamespaceExtension(...)