azure-rest-api-specs icon indicating copy to clipboard operation
azure-rest-api-specs copied to clipboard

Update types.json

Open dolauli opened this issue 1 year ago • 4 comments

Readonly should be set in properties

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

dolauli avatar Apr 12 '24 03:04 dolauli

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ This PR has at least one breaking change (label: BreakingChangeReviewRequired).
    To unblock this PR, follow the process at aka.ms/brch.
  • ❌ The required check named Swagger BreakingChange has failed. To unblock this PR, follow the process at aka.ms/brch.

Swagger Validation Report

️❌BreakingChange: 12 Errors, 0 Warnings failed [Detail]
Compared specs (v0.10.7) new version base version
types.json 4.0(75118b3) 4.0(main)
Rule Message
1029 - ReadonlyPropertyChanged The read only property has changed from 'true' to 'false'.
New: common-types/resource-management/v5/types.json#L30:9
Old: common-types/resource-management/v5/types.json#L30:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v5/types.json#L506:9
Old: common-types/resource-management/v5/types.json#L507:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v5/types.json#L511:9
Old: common-types/resource-management/v5/types.json#L511:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v5/types.json#L526:9
Old: common-types/resource-management/v5/types.json#L525:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v5/types.json#L532:9
Old: common-types/resource-management/v5/types.json#L530:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v5/types.json#L537:9
Old: common-types/resource-management/v5/types.json#L534:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'false' to 'true'.
New: common-types/resource-management/v5/types.json#L552:9
Old: common-types/resource-management/v5/types.json#L548:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'true' to 'false'.
New: common-types/resource-management/v5/types.json#L30:9
Old: common-types/resource-management/v5/types.json#L30:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'true' to 'false'.
New: common-types/resource-management/v5/types.json#L30:9
Old: common-types/resource-management/v5/types.json#L30:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'true' to 'false'.
New: common-types/resource-management/v5/types.json#L30:9
Old: common-types/resource-management/v5/types.json#L30:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'true' to 'false'.
New: common-types/resource-management/v5/types.json#L30:9
Old: common-types/resource-management/v5/types.json#L30:9
1029 - ReadonlyPropertyChanged The read only property has changed from 'true' to 'false'.
New: common-types/resource-management/v5/types.json#L502:5
Old: common-types/resource-management/v5/types.json#L502:5
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️❌SwaggerAPIView: 1 Errors, 0 Warnings failed [Detail]
Rule Message
Failed to generate swagger APIView. The readme file format is invalid and the tag is not defined. Use the provided readme template for guidance readme template. For more details, please check the detail log. "How to fix":"Check the readme file and make sure the readme file format is valid and the tag is defined. Use the provided readme template"
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]
 Please click here to preview with your @microsoft account. 
Posted by Swagger Pipeline | How to fix these errors?

PR validation pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Hi, @dolauli. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity label.

Hi, @dolauli. The PR will be closed since the PR has no update for 28 days. If you still need the PR review to proceed, please reopen it and @ mention PR assignee.