googleapis / googleapis/google-cloud-java

[java-shared-config] Checkstyle shoud set either `failOnViolation` or `failOnError` to `true`

Aberta
#13,751 0 comentários 0 reações 0 responsáveis Ver no GitHub
type: feature request
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

Abrir o 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

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.