Skip to content

resolved issue 471#486

Merged
tiagornandrade merged 8 commits intobrazilian-utils:mainfrom
tiagornandrade:471
Jan 31, 2025
Merged

resolved issue 471#486
tiagornandrade merged 8 commits intobrazilian-utils:mainfrom
tiagornandrade:471

Conversation

@tiagornandrade
Copy link
Copy Markdown
Contributor

Descrição

Este Pull Request melhora a visibilidade dos erros de lint no workflow Check Lint. Agora, quando o job de lint falhar, os detalhes dos erros serão exibidos diretamente no log do GitHub Actions, permitindo que os colaboradores identifiquem e corrijam os problemas sem precisar rodar localmente.

Mudanças Propostas

  • Captura a saída do make check e exibe os detalhes dos erros diretamente no log do GitHub Actions.
  • Substitui continue-on-error: true para garantir que os erros sejam mostrados antes da falha.
  • Melhora a mensagem de erro para indicar claramente como corrigir os problemas.

Checklist de Revisão

Comentários Adicionais (opcional)

Essa melhoria facilita a colaboração, reduzindo a necessidade de rodar o lint localmente para entender os erros.

Issue Relacionada

Closes #471

@tiagornandrade tiagornandrade requested review from a team as code owners January 31, 2025 14:35
Copy link
Copy Markdown
Member

@antoniamaia antoniamaia left a comment

Choose a reason for hiding this comment

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

Boa!! Obrigada, @tiagornandrade 🚀

@tiagornandrade tiagornandrade merged commit b8e84bd into brazilian-utils:main Jan 31, 2025
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.

Melhorar a saída de erro do lint no workflow para facilitar correções

2 participants