futureverse / futureverse/progressify
Add support for for-loops
- Dominant language
- R
- Stars
- 4
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
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)
})
```
Contributor guide
No contributing guide indexed for this repository
Research direction
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.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100