JuliaPluto/Pluto.jl

ExpressionExplorer: `global` stronger than `let`

Open

#1 856 ouverte le 26 janv. 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Julia (329 forks)batch import
backendbugexpression explorergood first issue

Métriques du dépôt

Stars
 (5 295 stars)
Métriques de merge PR
 (Merge moyen 5j 12h) (13 PRs mergées en 30 j)

Description

We have a couple of combinations of let and global that are not analyzed correctly, like:

let
	a = 1
	global a = 2
end

defines a global a, but Pluto prioritizes the let block.

@test_broken means that we still need to fix it:

https://github.com/fonsp/Pluto.jl/blob/c1d23122845e7864ef3dfe15a8428366c98f95eb/test/ExpressionExplorer.jl#L368-L379

This is a good first issue, because the ExpressionExplorer is a small, isolated part of Pluto's codebase. For more info about working on this issue, see:

https://github.com/fonsp/Pluto.jl/blob/c1d23122845e7864ef3dfe15a8428366c98f95eb/test/ExpressionExplorer.jl#L3-L17

Guide contributeur

Direction de recherche
Examinez le fichier `test/ExpressionExplorer.jl` autour des lignes 368-379 où `@test broken` est utilisé. Comprenez le comportement de portée attendu et identifiez le code dans ExpressionExplorer qui traite les instructions `let` et `global`, puis implémentez la correction et décommentez le test.
Stack technique
Aucun
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JuliaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80