uber/piranha

PiranhaJava - Doesn't add required space while removing parentheses

Aperta

#37 aperta il 14 mag 2020

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)Java (178 fork)batch import
good first issuelegacy-java

Metriche repository

Star
 (2143 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina la trasformazione removeParentheses o logica simile per le dichiarazioni return e aggiungi una corretta gestione degli spazi.
Tech stack
java
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Java
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75