grpc/grpc-java

Recommended IntelliJ code style settings are out of date

Open

#617 aberto em 10 de jul. de 2015

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Java (3.742 forks)batch import
help wanted

Métricas do repositório

Stars
 (11.124 stars)
Métricas de merge de PR
 (Mesclagem média 23d 22h) (43 fundiu PRs em 30d)

Description

I downloaded and imported the GoogleStyle code style settings xml, as recommended in Contributing.md. It doesn't seem like this project and its checkstyle are actually conforming to those style settings (which is fine, I like the prevailing gRPC style better!) A few things in particular keep coming up:

  • Import order: GoogleStyle has static imports at the bottom, doesn't separate grpc imports. This is configured in Code Style > Java > Imports > Import Layout.
  • Method argument wrapping: GoogleStyle has method call arguments and declaration parameters on the next line aligned to the opening paren. This is configured in Code Style > Java > Wrapping and Braces > Method declaration parameters > Align when multiline, as well as the same option under Method call arguments.

Guia do colaborador

Direção de pesquisa
Atualize o arquivo de configuração de estilo de código do IntelliJ para corresponder à configuração do checkstyle do projeto.
Pilha de tecnologia
java
Domain
developer experience
Tipo Issue
Documentação
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80