futureverse / futureverse/progressify

Add support for for-loops

Abierto
#2 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
R
Estrellas
4
Forks
0
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
r
Área
tooling
Tipo de issue
Nueva funcionalidad
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.