False positives when readOnly property using referenced enum
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 42/100
Direzione di ricerca
Inizia ispezionando il confronto descritto in Azure/azure-rest-api-specs-pr#15631, usando il modello Trial e l’enum TrialStatus referenziato come caso di riproduzione. Traccia il punto in cui il diff confronta lo schema enum inline precedente con il nuovo schema $ref e il valore readOnly. L’attività è completata quando questa modifica equivalente non viene più segnalata come modifica breaking di readOnly.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the bug
I'm working on https://github.com/Azure/azure-rest-api-specs-pr/pull/15631 and there are a lot of false positives with a message of `The read only property has changed from 'true' to 'false'.
To Reproduce
As an example from that PR:
@doc("Subscription trial availability")
model Trial {
@doc("Trial status")
@visibility("read")
status?: TrialStatus;
It does produce readOnly.
"status": {
"$ref": "#/definitions/TrialStatus",
"description": "Trial status",
"readOnly": true
},
But before it was not a $ref, it was inline:
"status": {
"description": "Trial status",
"type": "string",
"enum": [
"TrialAvailable",
"TrialUsed",
"TrialDisabled"
],
"readOnly": true,
"x-ms-enum": {
"name": "TrialStatus",
"modelAsString": true
}
},
Expected behavior
It is not marked as a breaking change.
- Lingua principale
- C#
- Stelle
- 290
- Fork
- 50
- Merge medio
- 2g 21h
- PR unite (30g)
- 10
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Azure/openapi-diff
-
Azure/openapi-diff#482 · 1 assegnatario ·
-
Azure/openapi-diff#479 · 2 commenti · 1 assegnatario ·
-
Remove dependency `autorest` Aperta
Azure/openapi-diff#456 · 1 assegnatario ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Azure/openapi-diff#445 ·
-
Azure/openapi-diff#371 · 7 commenti · 1 assegnatario ·
Tutte le issue di Azure/openapi-diff
Issue simili
-
Documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
cake-build/cake#5024 ·
-
Frontend status/draft TechnicalDebt
Difficoltà 2/5 1-2 giorni Idoneità per principianti 75/100
Altinn/altinn-auth#4143 ·
-
.NET Flaky Test Testing Tests
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
getsentry/sentry-dotnet#5617 · 1 commento ·
-
:watch: Not Triaged dotnet-fsharp/svc
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
Client customer-reported needs-team-attention question Service Attention WebPubSub
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Azure/azure-sdk-for-net#63292 · 3 commenti · 1 reazione ·