googleapis / googleapis/google-cloud-java
[java-shared-config] Checkstyle shoud set either `failOnViolation` or `failOnError` to `true`
- Linguagem predominante
- Java
- Estrelas
- 2.1k
- Forks
- 1.2k
- Merge médio
- 1d 23h
- PRs com merge (30d)
- 139
Descrição
Currently, both `failOnViolation` and `failOnError` are set (https://github.com/googleapis/java-shared-config/blob/main/pom.xml#L649,L651).
The way I am reading checkstyle documentation, they are more either/or -- either fail before logging or log and then fail: https://maven.apache.org/plugins/maven-checkstyle-plugin/check-mojo.html.
Thanks @elefeint for pointing this out.
Guia de contribuição
Direção de pesquisa
Comece por java-shared-config/pom.xml, na configuração do Checkstyle por volta das linhas 649 e 651, e depois leia a documentação do Maven Checkstyle plugin vinculada na issue. Confirme como as duas configurações devem interagir e atualize a configuração para que o requisito de ou uma ou outra da issue seja atendido.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- java
- Domínio
- build-system
- Tipo de issue
- Bug
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 42/100