Skip to content

Sobre o projeto: #1

@fandredev

Description

@fandredev
  • A classe SignerRequest aparentemente são para fazer requisições para coisas relacionadas a signatários. As funções get_api_token e set_api_token são funções para token de API e não tem relação direta com Signer. Essas duas funções não deveriam estar nessa classe, pois ferem o S de SOLID, onde cada classe deve ter a sua função e nada além disso.

image

  • Quando usamos fstrings, sempre vamos retornar str como tipo no Python, então não precisariamos tipar aqui (não é um problema tipar)

image

  • Adorei o uso do import de tipagem além das padrões de qualquer linguagem (bool, str, int) em doc_response.py
    image

  • Os builders ficaram muito bons também

  • Senti falta do requirements.txt pra gente saber das libraries do projeto e também de uma pasta venv pra usar o ambiente virtual do Python na raiz do projeto

  • Aqui o api_route poderia ficar no .env. Se tivessemos mais requests, teriamos que repetir o api_route nas outras classes e isso no .env ficaria centralizado caso mudasse alguma hora a URL.
    image

No geral, gostei muito e ficou claro que você sabe escrever código em Python bem suave kkkkkk, parabéns thay você é braba

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions