doc: add the documents for state holidays#545
doc: add the documents for state holidays#545VictorVendrera wants to merge 1 commit intoBrasilAPI:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Kudos, SonarCloud Quality Gate passed!
|
RodriAndreotti
left a comment
There was a problem hiding this comment.
Esse PR depende de algum outro PR para ser poder subir?
| }, | ||
| "name": { | ||
| "type": "string", | ||
| "format": "int32" |
There was a problem hiding this comment.
int32 não deveria ser usado só para tipos numéricos?
|
Olá @VictorVendrera, documentar endpoints é muito bem-vindo! Mas há um problema: este PR documenta o endpoint A prática no projeto é que a documentação OpenAPI acompanhe o PR de implementação — não ser separada. Isso garante que docs e código permanecem sincronizados. Sugestão: aguardar a implementação ser mergeada (ex: PR #552 que implementa feriados estaduais) e então contribuir com docs, ou abrir um PR que inclua tanto a implementação quanto a documentação. Outros ajustes necessários:
|
No description provided.