F542 is not a crime
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza localizando la regla F542 y sus pruebas existentes en pyflakes; después, revisa los ejemplos enlazados de psycopg tests/test_tstring.py que muestran t-strings válidos sin marcadores de posición. Se considerará terminado decidir si F542 debe deshabilitarse de forma predeterminada y añadir cobertura para el comportamiento elegido, incluida la salida de lint correspondiente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hello,
I am working on adding template strings support to psycopg. In the tests, Flake8 7.3.0 raises several warnings about template strings not containing placeholders (F542).
Unlike f-strings, a missing placeholder shouldn't be a problem. In the first iterations of PEP 750 it was possible to mix normal and t-strings with the same ease in which f-strings and normal strings can be mixed, but this feature has been identified as problematic and dropped from the design and latest Python implementation (post alpha and into beta, I seem to understand). As a consequence, F542 is not immediately fixable by dropping the t prefix and it should be normal to assemble a t-string by concatenating t-strings, some of which may not include placeholder as in this snippet:
t = t"select " # noqa: F542
for i, name in enumerate(("foo", "bar", "baz")):
if i:
t += t", " # noqa: F542
t += t"{i} as {name:i}"
Would you consider disabling F542 by default?
- Lenguaje dominante
- Python
- Estrellas
- 1.5k
- Forks
- 205
- Merge medio
- 1 d 2 h
- PR fusionados (30 d)
- 5
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 PyCQA/pyflakes
-
setuptools deprecation warningsAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 52/100
-
Use of await in a function that is not async should be shown as a syntax error by flake8Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
F821 with Annotated import aliasQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Todos los issues de PyCQA/pyflakes
Issues similares
-
ChatBedrockConverse changes the keys of JSON tool results to camelCase (`order_id` becomes `orderId`)Posiblemente ocupada @geyazheng la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
langchain-ai/langchain-aws#1469 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
EESSI/docs#949 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
b4b usability
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
fix(schedule): align list_tasks since cursor filter boundary from strict > to >=Posiblemente ocupada @bfs2021 la tomó hoy. Abiertoagent-ready bug p2 schedule
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
allenporter/supernote#540 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
frazane/scoringrules#164 ·
Los mantenedores suelen responder en 1 día