Rename `fresh` to more useful name
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 65/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- java
- Domaine
- developer-experience
Piste de recherche
Commencez par repérer où les résultats des conditions de chemin sont attribués à des variables nommées freshX et examinez l’exemple présenté dans l’issue. Choisissez un nom plus clair, comme then_branch ou then, qui identifie de manière cohérente la condition de chemin. Le travail est terminé lorsque les noms des variables générées concernées n’utilisent plus fresh et que l’exemple reflète cette dénomination plus claire.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
When we add a path condition, the variable getting that result is called freshX.
That works internally but it doesn't resonate a lot for a developer to understand thats a path condition.
We can use better names like then_branch or then, or similar ones, to clearly identify what the variable represents.
In the example below we can change the fresh to one of those solutions
- Langage dominant
- Java
- Étoiles
- 67
- Forks
- 36
- Merge moyen
- 2 j 3 h
- PR mergées (30 j)
- 29
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de liquid-java/liquidjava
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
liquid-java/liquidjava#321 ·
Les mainteneurs répondent en général sous 1 jour
-
Synthesize hints for resolutionOuverteenhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
liquid-java/liquidjava#381 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement future latte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
liquid-java/liquidjava#380 ·
Les mainteneurs répondent en général sous 1 jour
-
An external spec cannot refine methods its class inheritsPeut-être pris @CatarinaGamboa l’a pris il y a 1 jour. Ouvertebug
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
liquid-java/liquidjava#379 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
liquid-java/liquidjava#378 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de liquid-java/liquidjava
Issues similaires
-
[Bug] The producer summary counts an unreported client version as a second version and warns about a version mixPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
apache/rocketmq-dashboard#6110 ·
Les mainteneurs répondent en général sous 4 jours
-
`Processing lsp` never exits and leaves orphaned processesPeut-être pris @overcast302 l’a pris aujourd’hui. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
processing/processing4#1578 · 1 commentaire ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
apache/doris-flink-connector#707 ·
-
ASM is not up-to-dateOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 60/100
Les mainteneurs répondent en général sous 1 jour
-
[BUG] S3 CORS responses omit Access-Control-Allow-Credentials for matched originsPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
floci-io/floci#5369 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour