JuliaPluto/Pluto.jl

ExpressionExplorer: `global` stronger than `let`

Open

#1.856 aberto em 26 de jan. de 2022

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Julia (5.295 stars) (329 forks)batch import
backendbugexpression explorergood first issue

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

Guia do colaborador

Pilha de tecnologia
None
Domain
tooling
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Julia basicslet vs global scoping
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80
Direção de pesquisa
Examine the `test/ExpressionExplorer.jl` file around lines 368-379 where `@test broken` is used. Understand the expected scoping behavior and identify the code in the ExpressionExplorer that handles `let` and `global` statements, then implement the fix and uncomment the test.