Skip to content

fs: Replace a bunch of uses of smol::fs with manual impls - #39906

Merged
osiewicz merged 9 commits into
mainfrom
fs-use-bg-executor
Oct 14, 2025
Merged

fs: Replace a bunch of uses of smol::fs with manual impls#39906
osiewicz merged 9 commits into
mainfrom
fs-use-bg-executor

Conversation

@osiewicz

@osiewicz osiewicz commented Oct 9, 2025

Copy link
Copy Markdown
Member

smol::fs uses a separate threadpool, which is a bit yuck.

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Oct 9, 2025
@osiewicz
osiewicz enabled auto-merge (squash) October 11, 2025 19:13
@osiewicz osiewicz self-assigned this Oct 13, 2025
@osiewicz
osiewicz merged commit 9ad6196 into main Oct 14, 2025
23 checks passed
@osiewicz
osiewicz deleted the fs-use-bg-executor branch October 14, 2025 10:38
osiewicz added a commit that referenced this pull request Oct 14, 2025
@osiewicz
osiewicz restored the fs-use-bg-executor branch October 14, 2025 10:42
osiewicz added a commit that referenced this pull request Oct 14, 2025
…40170)

Reverts #39906

This PR should not have landed prior to Wednesday.
jasonsmithio pushed a commit to paddleboarddev/paddleboard that referenced this pull request May 31, 2026
…40170)

Reverts zed-industries/zed#39906

This PR should not have landed prior to Wednesday.
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…ries#39906)

smol::fs uses a separate threadpool, which is a bit yuck.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant