False positivizes and undetailed errors for RequiredStatusChange
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 30/100
Rechercherichtung
Beginne damit, die in breaking-changes.json angezeigten RequiredStatusChange-Einträge über das openapi-diff-Befehlszeilentool zu reproduzieren, und verfolge anschließend die Vergleichslogik, die diese Meldungen erzeugt. Erledigt ist die Aufgabe, wenn die gemeldeten Änderungen erforderlicher Werte korrekt sind und die aufgeführten False Positives nicht mehr auftreten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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 |
- Vorherrschende Sprache
- C#
- Sterne
- 290
- Forks
- 50
- Ø Merge
- 7 Std. 8 Min.
- Gemergte PRs (30 T.)
- 11
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Azure/openapi-diff
-
Crash if inherited property is structurally equal but not reference identicalEvtl. wieder frei @mikeharder hat das vor 152 Tagen übernommen, und es ist kein Pull Request offen. Offen
Azure/openapi-diff#482 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Crash if duplicate model names (across files) used in allOfEvtl. wieder frei @mikeharder hat das vor 153 Tagen übernommen, und es ist kein Pull Request offen. Offen
Azure/openapi-diff#479 · 2 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Remove dependency `autorest`Evtl. wieder frei @mikeharder hat das vor 215 Tagen übernommen, und es ist kein Pull Request offen. Offen
Azure/openapi-diff#456 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
Azure/openapi-diff#445 ·
Maintainer antworten meist innerhalb von 1 Tag
-
OpenAPI-diff should flag changes in the tags for an operationEvtl. wieder frei @mikeharder hat das vor 398 Tagen übernommen, und es ist kein Pull Request offen. Offen
Azure/openapi-diff#371 · 7 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in Azure/openapi-diff
Ähnliche Issues
-
[C#]:主页联网更新的提示投稿横幅指向错误Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
PCL-Community/PCL-CE#3652 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:frontend bug FE P3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
klasolsson81/jobbliggaren#2010 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[aw] Upgrade availableOffenagentic-workflows untriaged
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100
Maintainer antworten meist innerhalb von 1 Tag
-
area: homeblaze type: bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
RicoSuter/Namotion.Interceptor#630 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Akka.Hosting enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100