Removing `readOnly: true` on response-only property is flagged as error
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
Comece revisando o commit de migração do Key Vault vinculado, o log de build do diff do OpenAPI e a thread de revisão do pull request referenciada para entender como as propriedades que aparecem apenas nas responses são comparadas. Reproduza o erro falso relatado e determine o resultado esperado para a remoção de readOnly: true; considera-se concluído quando o diff não sinalizar mais essa alteração válida como um erro, mas continuar detectando breaking changes reais.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
While migrating Key Vault specs from Swagger to TSP, we removed a readOnly designation for a property on a response-only type. This was flagged as an error by the OpenAPI diff; per context in this review thread, this error doesn't seem to be correct.
- Linguagem predominante
- C#
- Estrelas
- 290
- Forks
- 50
- Merge médio
- 7h 8min
- PRs com merge (30d)
- 11
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de Azure/openapi-diff
-
Crash if inherited property is structurally equal but not reference identicalTalvez livre de novo @mikeharder assumiu há 158 dias e não há nenhum pull request aberto. Aberta
Azure/openapi-diff#482 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Crash if duplicate model names (across files) used in allOfTalvez livre de novo @mikeharder assumiu há 159 dias e não há nenhum pull request aberto. Aberta
Azure/openapi-diff#479 · 2 comentários · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Remove dependency `autorest`Talvez livre de novo @mikeharder assumiu há 221 dias e não há nenhum pull request aberto. Aberta
Azure/openapi-diff#456 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
Azure/openapi-diff#445 ·
Mantenedores costumam responder em até 1 dia
-
OpenAPI-diff should flag changes in the tags for an operationTalvez livre de novo @mikeharder assumiu há 404 dias e não há nenhum pull request aberto. Aberta
Azure/openapi-diff#371 · 7 comentários · 1 responsável ·
Mantenedores costumam responder em até 1 dia
Todas as issues de Azure/openapi-diff
Issues semelhantes
-
再現済み 要トリアージ 誤判定
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
yksr-melt/Meltype#421 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 64/100
Facepunch/sbox-public#12063 · 1 comentário ·
Mantenedores costumam responder em até 2 dias
-
documentation
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
facioquo/stock-indicators-dotnet#2316 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
eriknihlen/OpenAC#219 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
ObsidianMC/Obsidian#548 ·
Mantenedores costumam responder em até 1 dia