Skip to content

feat: add internal structures#4

Draft
ortense wants to merge 3 commits intomainfrom
feat/get-by-tag-name
Draft

feat: add internal structures#4
ortense wants to merge 3 commits intomainfrom
feat/get-by-tag-name

Conversation

@ortense
Copy link
Member

@ortense ortense commented Apr 1, 2024

Este PR adiciona as funcionalidades para obter o status de uma tag pelo seu nome

WIP

  • structs e pacotes para client e tag
  • implementação de repositório em memória para teste
  • criação do usecase para obter a tag
    • testes de caminho feliz
    • testes de caso de erro
  • conexão com redis
  • criação do repositório com redis
  • validar api key
  • expor api rest

@ortense ortense force-pushed the feat/get-by-tag-name branch from 895801c to 672bb79 Compare April 1, 2024 23:13
return &value, nil
}

return nil, nil

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

talvez aqui retornar o &value, nil e todas validacoes acima forem cenario de error, me estranhou o "nil, nil"

ou é nil, value | value,nil

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não achar uma tag não é um erro, ele só não existe, o repo fez a parte dele que era buscar pelo nome 🙃

Se não existe, nem deu erro return nil, nil

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants