api-version should not cause a ChangedParameterOrder error

Abierto

#287 abierto el 17/11/2023 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 3/5
Tiempo estimado: 1-2 días
Aptitud para principiantes: 45/100

Tipo de issue: Error
Claridad: Bastante claro
Estado de actividad: Estancado
Stack tecnológico: csharp
Área: api, cli

Línea de trabajo

Busca en el repositorio la regla ChangedParameterOrder y cómo gestiona las referencias a parámetros; después, compara las listas de parámetros original y actualizada que se muestran en la issue. Se considera terminado cuando reordenar ApiVersionParameter ya no produce este error de breaking change, mientras que los demás cambios en el orden de los parámetros siguen detectándose; valídalo con el ejemplo de la issue y las comprobaciones relevantes del proyecto.

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 . I have 57 ChangedParameterOrder breaking change errors. Many are due to ApiVersionParameter changing order. It should be ignored by the ChangedParameterOrder rule.

ChangedParameterOrder

This is the original.

        "operationId": "Locations_CheckQuotaAvailability",
        "parameters": [
          {
            "$ref": "../../../../../common-types/resource-management/v3/types.json#/parameters/SubscriptionIdParameter"
          },
          {
            "name": "location",
            "in": "path",
            "required": true,
            "type": "string",
            "description": "Azure region"
          },
          {
            "$ref": "../../../../../common-types/resource-management/v5/types.json#/parameters/ApiVersionParameter"
          }
        ],

To:

        "operationId": "Locations_CheckQuotaAvailability",
        "parameters": [
          {
            "$ref": "../../../../../common-types/resource-management/v3/types.json#/parameters/ApiVersionParameter"
          },
          {
            "$ref": "../../../../../common-types/resource-management/v3/types.json#/parameters/SubscriptionIdParameter"
          },
          {
            "name": "location",
            "in": "path",
            "description": "A location in a subscription",
            "required": true,
            "type": "string",
            "pattern": "^[-\\w\\._]+$"
          }
        ],

If I move the ApiVersionParameter in the original version to be first, the rule does not consider it breaking.

Lenguaje dominante: C#
Estrellas: 290
Forks: 50
Merge medio: 2 d 21 h
PR fusionados (30 d): 10

Guía de contribución

Abrir la 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

Crash if inherited property is structurally equal but not reference identical Abierto

Azure/openapi-diff#482 · 1 asignado · Abierto el 5/5/2026
Crash if duplicate model names (across files) used in allOf Abierto

Azure/openapi-diff#479 · 2 comentarios · 1 asignado · Abierto el 5/5/2026
Remove dependency `autorest` Abierto

Azure/openapi-diff#456 · 1 asignado · Abierto el 3/3/2026
Error: Invalid reference token: /Tables(\'{tableName}\') Abierto

Dificultad 4/5 3-5 días Aptitud para principiantes 35/100

Azure/openapi-diff#445 · Abierto el 27/1/2026
OpenAPI-diff should flag changes in the tags for an operation Abierto

Azure/openapi-diff#371 · 7 comentarios · 1 asignado · Abierto el 1/9/2025

Todos los issues de Azure/openapi-diff

Issues similares

Add Cake DSL examples to BuildSystem and CI provider APIs Abierto

Documentation

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

cake-build/cake#5024 · Abierto el 23/9/2026
Gå gjennom ESLint-suppressions Abierto

Frontend status/draft TechnicalDebt

Dificultad 2/5 1-2 días Aptitud para principiantes 75/100

Altinn/altinn-auth#4143 · Abierto el 23/9/2026
Flaky: msbuild integration test times out flushing at process exit (net9.0, Windows) Abierto

.NET Flaky Test Testing Tests

Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100

getsentry/sentry-dotnet#5617 · 1 comentario · Abierto el 23/9/2026
Add more to the documentation Abierto

:watch: Not Triaged dotnet-fsharp/svc

Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100

dotnet/docs#56209 · Abierto el 22/9/2026
WebPubSubServiceClient.GetClientAccessUri (sync) drops the groups argument on the TokenCredential path Abierto

Client customer-reported needs-team-attention question Service Attention WebPubSub

Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100

Azure/azure-sdk-for-net#63292 · 3 comentarios · 1 reacción · Abierto el 22/9/2026

Más issues de C#