python / python/cpython

Confusing syntax error for missing comma after import-as target

Abierto
#146,492 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

3.14 3.15 interpreter-core topic-parser type-bug
Lenguaje dominante
Python
Estrellas
77.2k
Forks
35.9k
Métricas de merge de PR
Métricas de PR pendientes

Descripción

Bug report

Bug description:

Right now we have:

>>> import a as a  b
  File "<python-input-0>", line 1
    import a as a  b
                ^
SyntaxError: cannot use name as import target

>>> from x import a as a  b
  File "<python-input-1>", line 1
    from x import a as a  b
                       ^
SyntaxError: cannot use name as import target

This happens since these cases match these two rules.

One possible fix would be to add NAME to the list of not-matched tokens so that the current rules don't match this case.

However, I have an idea for a slightly nicer error. PR forthcoming.

cc @pablogsal @lysnikolaou @sobolevn

CPython versions tested on:

3.14, 3.15, CPython main branch

Operating systems tested on:

Linux

Linked PRs
  • gh-146493

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Reproduce los dos ejemplos de import-as en la Python REPL y, después, inspecciona las dos reglas coincidentes en Grammar/python.gram alrededor de las líneas 1435 y 1439. Compara el diagnóstico resultante con el comportamiento esperado descrito en el issue; se considera terminado cuando el caso de la coma faltante recibe un error de sintaxis más claro. Ya se ha indicado un PR enlazado.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
compilers
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.