Add @access and @usage support for named union
#413
Conversation
tadelesh
requested review from
lmazuel,
m-nash,
iscai-msft,
srnagar,
joheredi,
bterlson,
markcowl,
allenjzhang and
timotheeguerin
as code owners
|
All changed packages have been documented. |
|
You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/413/ Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/413/ |
| @@ -92,6 +92,8 @@ extern dec client(target: Namespace | Interface, value?: {}, scope?: valueof str | |||
| extern dec operationGroup(target: Namespace | Interface, scope?: valueof string); | |||
|
|
|||
| /** | |||
| * DEPRECATED: Use `@usage` and `@access` decorator instead. | |||
There was a problem hiding this comment.
Do you want to mark them explicitly as deprecated instead of just a comment that most people don't read?
There was a problem hiding this comment.
@timotheeguerin they've already been previously marked as deprecated. They've been deprecated for a while, is it fair for us to remove support for it now?
There was a problem hiding this comment.
we have still never removed any deprecated element from typespec but for tcgc probably ok if you do as it shouldn't be used in RPSaaSmaster yet and otherwise it would have already failed
resolve: #403
Also add deprecation doc.