MagicStack / MagicStack/MagicPython

Add scope for function parameter used inside function

Abierto
#208 1 comentario 2 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
JavaScript
Estrellas
1.4k
Forks
98
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Got send here from 'vscode-python' https://github.com/microsoft/vscode-python/issues/11594
```
def function(param1):
x = 1
return param1 * x
```
Both `x` and `param1` are variables used inside the `function`, but `param1` is a given parameter.

In the function def line `param1` has a special scope but not when used inside the function.
I'd like to have it have a special textmate scope too so I can color it and see, when I use a parameter and when a 'normal' variable.

The left side is what I want (from PyCharm) and the right side is as currently is.
![image](https://user-images.githubusercontent.com/43916661/81099878-e05fac00-8f0b-11ea-9b84-dd623270aa8b.png)

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

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

El issue no menciona archivos de implementación ni tests; empieza localizando las reglas de resaltado de sintaxis de Python de MagicPython y comparando la definición del parámetro del ejemplo de función con su uso dentro de la función. La tarea estará terminada cuando las referencias a parámetros reciban un scope de TextMate distinto del de las variables locales ordinarias y exista cobertura para el ejemplo mostrado.

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

Evaluación

Stack tecnológico
javascript, python
Área
devtools, tooling
Tipo de issue
Nueva funcionalidad
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
38/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.