Skip to content

Conversation

@Douglas019BR
Copy link
Contributor

Solve the issue :

#63

@Douglas019BR Douglas019BR force-pushed the complete_cpf_with_zeros_issue#63 branch from d4e5910 to cea87d8 Compare June 6, 2024 17:45
@Douglas019BR Douglas019BR requested a review from alvarofpp July 31, 2024 17:54
@alvarofpp
Copy link
Owner

@Douglas019BR O PR falhou no lint:

  • O primeiro commit deveria ter letra maiúscula após os :.
  • O segundo commit deveria estar de acordo com o Conventional Commits.

@Douglas019BR Douglas019BR force-pushed the complete_cpf_with_zeros_issue#63 branch from e6865a7 to fa2b910 Compare August 10, 2024 13:35
@Douglas019BR
Copy link
Contributor Author

Agora os commits estao de acordo com o Conventional Commits.
Obrigado pelo apontamento.

@alvarofpp
Copy link
Owner

@Douglas019BR Acredito que dessa vez seja o espaço em branco faltante entre o type e o subject.

  • Errado: feat(issue63):Complete ...
  • Certo: feat(issue63): Complete ...

O repositório consta com um Makefile com comandos para facilitar o desenvolvimento. Para fazer essa mesma checagem do lint localmente, basta executar make lint. Depois vem o job de testes, nesse caso execute make test e make test-coverage (para esse caso é necessário ter a imagem do projeto, portanto execute make build antes dos comandos de testes).

@Douglas019BR Douglas019BR force-pushed the complete_cpf_with_zeros_issue#63 branch from fa2b910 to a3d25d1 Compare August 14, 2024 13:14
@Douglas019BR
Copy link
Contributor Author

Douglas019BR commented Aug 14, 2024

@alvarofpp , me perdoe, eu fiz as alteracoes com pressa e nao me atentei ao Makefile, erro meu total.
Agora revisitei tudo com mais calma, melhorei os testes tambem, e rodei cada etapa do make localmente, acredito que esteja tudo correto.
Parabens pela organizacao do Makefile e o hook, gostei bastante de como organizou.

image

acredito que o problema do lint seja no markdown dessa vez:
image

@alvarofpp
Copy link
Owner

Aparentemente o arquivo .lint/.markdownlintrc não está sendo reconhecido, esse erro é comigo. Irei corrigir esse workflow e executar novamente para a sua branch.

Copy link
Owner

@alvarofpp alvarofpp left a comment

Choose a reason for hiding this comment

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

Obrigado pela sua contribuição!

@alvarofpp alvarofpp merged commit 7c2fef7 into alvarofpp:main Aug 14, 2024
@alvarofpp
Copy link
Owner

@Douglas019BR A sua modificação será lançada na versão 2.0.0. Abri uma issue para listar as atualizações que irei realizar para a versão 2.0.0: #67. Sinta-se a vontade para fazer qualquer sugestão nessa issue.

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.

2 participants