JuliaPluto/Pluto.jl

ExpressionExplorer: `global` stronger than `let`

Open

Aperta il 26 gen 2022

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Julia (5295 star) (329 fork)batch import
backendbugexpression explorergood first issue

Descrizione

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

Guida contributor

Tech stack
Nessuno
Dominio
tooling
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Julia basicslet vs global scoping
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80
Direzione di ricerca
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.