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

Adiciona erro a ser ignorado #22

Merged
merged 2 commits into from
Jan 23, 2020
Merged

Adiciona erro a ser ignorado #22

merged 2 commits into from
Jan 23, 2020

Conversation

aguiarcandre
Copy link
Contributor

Adicionei um comentário com breve explicação do erro e adicionei o código E203 a ser ignorado.

Seria interessante explicitar o código do erro nos comentários? Para ser possível identificar rapidamente em um cenário com diversos erros.

@daltonmatos
Copy link
Member

Legal!! Acho que vale até adicionar os erros em cada um dos comentários. O que acha?
Essa é a lista completa de erros: https://pep8.readthedocs.io/en/release-1.7.x/intro.html#error-codes

Dessa forma so de olhar o arquivo já conseguiremos saber o que cada erro significa.

O que acha?

@aguiarcandre
Copy link
Contributor Author

Concordo, realmente facilitaria muito em um cenário com diversos erros! Já vou alterar 😄

@daltonmatos
Copy link
Member

Agora que vi que o CI estava configurado para não fazer build de PR vindos de forks. Mudei a config mas pra esse PR só vai rodar o pipeline no próximo commit. Vou remover a configuração de merge checks só pra fazer esse merge e depois ligo novamente.

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

Successfully merging this pull request may close these issues.

2 participants