uber/piranha

PiranhaJava - Doesn't add required space while removing parentheses

Aberta

#37 aberto em 14 de mai. de 2020

Ver no GitHubGuia do colaborador
 (6 comentários) (0 reação) (0 responsável)Java (178 forks)batch import
good first issuelegacy-java

Métricas do repositório

Stars
 (2.143 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Piranha changed return(true); to returntrue;. This is sort of related to #35 - in the sense that PiranhaJava doesn't validate the parens/spaces of the output generated.

This error probably would not have occured if the code had been Google Java formatted.

However, I thought I will bring it to your attention, since this is a case of Piranha introducing a "bug" instead of erring on the side of caution, like #35

Guia do colaborador

Direção de pesquisa
Investigue a transformação removeParentheses ou lógica semelhante para declarações return e adicione o tratamento adequado de espaços.
Pilha de tecnologia
java
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Java
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75