Skip to content

Commit 2f9173d

Browse files
mcgallanpshao25
andauthored
Remove Private Decorator in StorageActions (#36905)
* update * remove suppression in model * removed suppression --------- Co-authored-by: Pan Shao <[email protected]>
1 parent 5726a14 commit 2f9173d

File tree

3 files changed

+8
-20
lines changed

3 files changed

+8
-20
lines changed

specification/storageactions/StorageAction.Management/StorageTask.tsp

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,8 @@ namespace Microsoft.StorageActions;
1313
/**
1414
* Represents Storage Task.
1515
*/
16-
#suppress "@azure-tools/typespec-azure-core/no-private-usage" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
17-
#suppress "@azure-tools/typespec-azure-core/no-private-usage" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
18-
#suppress "@azure-tools/typespec-azure-core/composition-over-inheritance" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
19-
@Azure.ResourceManager.Private.armResourceInternal(StorageTaskProperties)
20-
@Http.Private.includeInapplicableMetadataInPayload(false)
21-
model StorageTask extends Foundations.TrackedResource {
16+
model StorageTask
17+
is Azure.ResourceManager.TrackedResource<StorageTaskProperties, false> {
2218
...ResourceNameParameter<
2319
Resource = StorageTask,
2420
KeyName = "storageTaskName",
@@ -31,12 +27,6 @@ model StorageTask extends Foundations.TrackedResource {
3127
*/
3228
#suppress "@azure-tools/typespec-azure-resource-manager/arm-resource-invalid-envelope-property" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
3329
identity: Azure.ResourceManager.Foundations.ManagedServiceIdentity;
34-
35-
#suppress "@azure-tools/typespec-azure-core/no-private-usage" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
36-
#suppress "@azure-tools/typespec-azure-resource-manager/arm-resource-invalid-envelope-property" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
37-
@doc("The resource-specific properties for this resource.")
38-
@Azure.ResourceManager.Private.armResourcePropertiesOptionality(false)
39-
properties?: StorageTaskProperties;
4030
}
4131

4232
@armResourceOperations

specification/storageactions/StorageAction.Management/models.tsp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,6 @@ model StorageTaskPreviewAction {
116116
/**
117117
* Properties of the storage task preview.
118118
*/
119-
#suppress "@azure-tools/typespec-azure-core/no-private-usage" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
120119
properties: StorageTaskPreviewActionProperties;
121120
}
122121

@@ -387,7 +386,6 @@ model StorageTaskUpdateParameters {
387386
/**
388387
* Properties of the storage task.
389388
*/
390-
#suppress "@azure-tools/typespec-azure-core/no-private-usage" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details"
391389
properties?: StorageTaskUpdateProperties;
392390
}
393391

specification/storageactions/resource-manager/Microsoft.StorageActions/stable/2023-01-01/storageactions.json

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -828,19 +828,19 @@
828828
"type": "object",
829829
"description": "Represents Storage Task.",
830830
"properties": {
831-
"identity": {
832-
"$ref": "../../../../../common-types/resource-management/v5/managedidentity.json#/definitions/ManagedServiceIdentity",
833-
"description": "The managed service identity of the resource."
834-
},
835831
"properties": {
836832
"$ref": "#/definitions/StorageTaskProperties",
837833
"description": "Properties of the storage task.",
838834
"x-ms-client-flatten": true
835+
},
836+
"identity": {
837+
"$ref": "../../../../../common-types/resource-management/v5/managedidentity.json#/definitions/ManagedServiceIdentity",
838+
"description": "The managed service identity of the resource."
839839
}
840840
},
841841
"required": [
842-
"identity",
843-
"properties"
842+
"properties",
843+
"identity"
844844
],
845845
"allOf": [
846846
{

0 commit comments

Comments
 (0)