`ConsoleOutputCapture` should toggle `warn` when the future is parked/reentered
Open
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 337
- Forks
- 32
- Avg merge
- 8d 17h
- Merged PRs (30d)
- 11
Description
Rather than toggled to warn=1 at the beginning, which lasts for the lifetime of the future, and then toggled to warn=old_warn at the end
See https://github.com/posit-dev/ark/pull/1222#discussion_r3543928956
Related to #1320, where we fixed a bug somewhat related to this
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating ConsoleOutputCapture and read the discussion in PR #1222, then trace how the future is parked and reentered. Verify that warn is toggled for each active period rather than for the future's entire lifetime, using any relevant existing tests to confirm the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100