JuliaPluto/Pluto.jl

ExpressionExplorer: `global` stronger than `let`

Aberta

#1.856 aberto em 26 de jan. de 2022

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)Julia (329 forks)batch import
backendbugexpression explorergood first issue

Métricas do repositório

Stars
 (5.295 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Direção de pesquisa
Examine o arquivo `test/ExpressionExplorer.jl` em torno das linhas 368-379 onde `@test broken` é usado. Compreenda o comportamento de escopo esperado e identifique o código no ExpressionExplorer que lida com as instruções `let` e `global`, então implemente a correção e remova o comentário do teste.
Pilha de tecnologia
None
Domain
backend
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 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JuliaGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80