Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pre-commit hook atualização de configurações #527

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

marcmatias
Copy link
Collaborator

Adicionado uma atualização de configurações para uso de pre-commit durante desenvolvimento.

Com a adição do pre-commit ao requirements-development.txt, após a instalação das dependências, é necessário apenas usar o comando :

pre-commit install

Para que as configurações do arquivo yaml sejam passadas para a pasta .git/hooks/pre-commit

docs/dev-process.md Outdated Show resolved Hide resolved
@rhenanbartels
Copy link
Contributor

Fala @marcmatias, baixei sua branch e testei o pre-commit. Parece que tudo funciona como esperado.

Com relação ao exclude no arquivo .pre-commit-config.yaml, o diretório node_modules pode ser retirado por conta da diretiva types de cada etapa, certo?

Fiz apenas um comentário sobre um typo no docs. No mais, vou marcar o PR como aprovado.

Abraços

@marcmatias
Copy link
Collaborator Author

Sim, o flake8 tem um arquivo de configuração separado o setup.cfg que aponta para essa pasta.

Rebase e correção de texto solicitada aplicada.

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.

None yet

2 participants