Change `return` to `$result` for method return values in refinements
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 25/100
Rechercherichtung
Beginne damit, die Refinement-Grammatik und den Code zu finden, der Rückgabewerte ersetzt oder prüft; das Issue erwähnt beides, nennt aber keine Dateien oder Tests. Vergleiche die bestehende Behandlung von return und _ mit der vorgeschlagenen Syntax $result und berücksichtige den offenen Pull Request, bevor du beginnst. Fertig ist die Aufgabe, wenn die neue Syntax funktioniert, ohne bestehendes Verhalten zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
return is a statement, and it's strange to have it as a variable to represent the return value.
Therefore, we could change it for result. But since result can be an actual variable that represents something other than the return value we, we can try using $result.
This would mean a small change in the grammar, probably, and a change in replacing the return value.
Ps: the _ should still work in any case
Example Before:
@Refinement("return > 0")
int test2 (){
return 10;
}
Example After:
@Refinement("$result > 0")
int test2 (){
return 10;
}
- Vorherrschende Sprache
- Java
- Sterne
- 67
- Forks
- 36
- Ø Merge
- 10 T. 18 Std.
- Gemergte PRs (30 T.)
- 3
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus liquid-java/liquidjava
-
enhancement
liquid-java/liquidjava#240 · 1 zugewiesene Person ·
-
enhancement ide
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
liquid-java/liquidjava#206 ·
-
tests
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 25/100
liquid-java/liquidjava#198 ·
-
tests
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
liquid-java/liquidjava#197 ·
-
tests
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
liquid-java/liquidjava#196 · 2 Kommentare ·
Alle Issues in liquid-java/liquidjava
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
elastic/gradle-plugins#157 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
cryptomator/hub#497 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
johanhaleby/occurrent#1120 ·