False positivizes and undetailed errors for RequiredStatusChange
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 30/100
Línea de trabajo
Comienza reproduciendo las entradas RequiredStatusChange que se muestran en breaking-changes.json mediante la herramienta de línea de comandos openapi-diff y, después, sigue la lógica de comparación que produce esos mensajes. Se considera terminado cuando los cambios de valores requeridos reportados son precisos y los falsos positivos enumerados ya no aparecen.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I'm working on https://github.com/Azure/azure-rest-api-specs-pr/pull/15631 and there are some required differences reported between the old API version and the generated from TypeSpec version regarding properties. The errors are not very accurate. They should say what required values changes. Many look to be false positives.
~/ms/azure-rest-api-specs-pr> open breaking-changes.json | where code == RequiredStatusChange | select message old.path | to md
| message | old_path | |
|---|---|---|
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.PrivateCloudList.properties | New list has "required": ["value"] |
| The 'required' status changed from the old version('True') to the new version('False'). | definitions.PrivateCloud.properties | Neither has required. |
| The 'required' status changed from the old version('True') to the new version('False'). | definitions.ManagementCluster.properties | Neither has required. |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.PrivateCloudIdentity.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.ClusterList.properties | |
| The 'required' status changed from the old version('True') to the new version('False'). | definitions.Cluster.properties | Neither has required. |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.DatastoreList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.HcxEnterpriseSiteList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.ExpressRouteAuthorizationList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.GlobalReachConnectionList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkSegmentsList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkDhcpList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkGatewayList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkPortMirroringList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkVMGroupsList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkVirtualMachinesList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkDnsServicesList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkDnsZonesList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.WorkloadNetworkPublicIPsList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.CloudLinkList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.AddonList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.VirtualMachinesList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.PlacementPoliciesList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.ScriptPackagesList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.ScriptCmdletsList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.ScriptExecutionsList.properties | |
| The 'required' status changed from the old version('False') to the new version('True'). | definitions.TrackedResource.properties |
- Lenguaje dominante
- C#
- Estrellas
- 290
- Forks
- 50
- Merge medio
- 2 d 21 h
- PR fusionados (30 d)
- 10
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Azure/openapi-diff
-
Azure/openapi-diff#482 · 1 asignado ·
-
Azure/openapi-diff#479 · 2 comentarios · 1 asignado ·
-
Remove dependency `autorest` Abierto
Azure/openapi-diff#456 · 1 asignado ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Azure/openapi-diff#445 ·
-
Azure/openapi-diff#371 · 7 comentarios · 1 asignado ·
Todos los issues de Azure/openapi-diff
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
nightscout/nocturne#1425 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
cake-build/cake#5024 ·
-
Gå gjennom ESLint-suppressions AbiertoFrontend status/draft TechnicalDebt
Dificultad 2/5 1-2 días Aptitud para principiantes 75/100
Altinn/altinn-auth#4143 ·