JuliaPluto/Pluto.jl

ExpressionExplorer: `global` stronger than `let`

Aperta

#1856 aperta il 26 gen 2022

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Julia (329 fork)batch import
backendbugexpression explorergood first issue

Metriche repository

Star
 (5295 stelle)
Metriche merge PR
 (Merge medio 5g 12h) (13 PR mergiate in 30 g)

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

Direzione di ricerca
Esamina il file `test/ExpressionExplorer.jl` intorno alle righe 368-379 dove viene utilizzato `@test broken`. Comprendi il comportamento di scope previsto e identifica il codice in ExpressionExplorer che gestisce le istruzioni `let` e `global`, quindi implementa la correzione e rimuovi il commento dal test.
Tech stack
Nessuno
Dominio
backend
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 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JuliaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80