futureverse / futureverse/progressify

Add support for for-loops

Aperta
#2 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
R
Stelle
4
Fork
0
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Add support to inject progress updates in for loops. For reasons explained in , it's impossible to support:

```r
for (x in xs) {
sqrt(x)
} |> progressify()
```

but should be able to support:

```r
( for (x in xs) {
sqrt(x)
} ) |> progressify()
```

and

```r
{ for (x in xs) {
sqrt(x)
} } |> progressify()
```

We can also support it without pipes;

```r
progressify(for (x in xs) {
sqrt(x)
})
```

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Start by locating the existing progressify implementation and how it handles piped expressions and function calls. Add coverage for parenthesized and braced for-loops, plus progressify(for (...) {...}), and verify that each form emits progress updates without supporting the explicitly excluded unwrapped pipe form.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
r
Ambito
tooling
Tipo di issue
Funzionalità
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.