Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Migrate file system operations to NIO #3076

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from
Draft

Migrate file system operations to NIO #3076

wants to merge 4 commits into from

Conversation

msfjarvis
Copy link
Member

No description provided.

@msfjarvis msfjarvis added C-technical-debt Category: This makes the code harder to read and modify, but has no impact on end users P-medium Priority: medium labels May 31, 2024
@msfjarvis msfjarvis added this to the v2.0.0 milestone May 31, 2024
@msfjarvis msfjarvis added the S-in-progress Status: Implementation is underway label May 31, 2024
@msfjarvis
Copy link
Member Author

The migration was mechanical and incorrectly assumes that listDirectoryEntries() will list child files recursively, all of this needs to be migrated to Path#walk

@msfjarvis msfjarvis force-pushed the hs/nio-paths branch 2 times, most recently from 7d7410b to 2c35189 Compare June 17, 2024 07:19
@msfjarvis msfjarvis force-pushed the hs/nio-paths branch 4 times, most recently from b65ec45 to 5db009e Compare July 22, 2024 20:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
C-technical-debt Category: This makes the code harder to read and modify, but has no impact on end users P-medium Priority: medium S-in-progress Status: Implementation is underway
Projects
APS v2 Backlog
Status: 👀 In review
Milestone
v2.0.0
Development

Successfully merging this pull request may close these issues.
1 participant
@msfjarvis