futureverse / futureverse/progressify

Add support for for-loops

Offen
#2 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
R
Sterne
4
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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)
})
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
r
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.