The BuilderPatternFunctionsRewriter class has some incorrect handling methods.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 38/100
Línea de trabajo
Comienza en la clase BuilderPatternFunctionsRewriter e inspecciona cómo gestiona los tipos de retorno de los métodos y las sentencias return. Reproduce el caso mostrado del getter estático y verifica después que solo se reescriban los métodos que devuelven this y que el código generado no tenga ninguna incompatibilidad de tipos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
orgin code is:
` private static ClassA staicField;
public static ClassA GetStaicField()
{
return staicField;
}`
will be written to
` private static ClassA staicField;
public static ClassA_Patched_0 GetStaicField()
{
return staicField;
}`
The above code may lead to type mismatch errors.
Only "return this" needs to be rewrite. Perhaps it is necessary to add a check for whether there is a "return this".
static bool ReturnsThis(MethodDeclarationSyntax method)
{
var returnStatements = method.DescendantNodes().OfType<ReturnStatementSyntax>();
return returnStatements.All(r => r.Expression is ThisExpressionSyntax);
}
- Lenguaje dominante
- C#
- Estrellas
- 2.2k
- Forks
- 169
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de handzlikchris/FastScriptReload
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
handzlikchris/FastScriptReload#202 · 2 reacciones ·
-
csc.rsp supportAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
-
Failed to find entry-pointsPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
handzlikchris/FastScriptReload#203 · 4 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
handzlikchris/FastScriptReload#197 · 1 comentario ·
Todos los issues de handzlikchris/FastScriptReload
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
lugassawan/swe-workbench#799 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
CommunityToolkit/AI#64 ·
-
Bug: Hidden loading rings keep the compositor running, so Files uses 7-94% of a CPU core while idleAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
files-community/Files#19026 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Physgun SFX SpamAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
Browser: atlas neighbours bleed at tile edges when minified (MSAA extrapolates UVs past the #315 inset) — make vTexCoord `centroid`Posiblemente ocupada @Anras573 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día