forked from bradygaster/squad
-
Notifications
You must be signed in to change notification settings - Fork 0
Track Wave 2: Remove deprecated sync StorageProvider methods #113
Copy link
Copy link
Open
Labels
go:needs-researchNeeds investigationNeeds investigationsquadSquad triage inbox — Lead will assign to a memberSquad triage inbox — Lead will assign to a membersquad:flightAssigned to Flight (Lead)Assigned to Flight (Lead)
Description
What
Remove the 12 deprecated synchronous methods from StorageProvider interface.
Why
Sync methods were kept in Wave 1 for backward compatibility. Wave 2 should remove them to simplify the interface and eliminate sync I/O paths.
Methods to Remove
readSync, writeSync, existsSync, listSync, isDirectorySync, mkdirSync, statSync, appendSync, deleteSync, renameSync, copySync, deleteDirSync
Tracking
- From FIDO's repo health review of PR feat(sdk): StorageProvider abstraction — complete migration + example providers bradygaster/squad#640
- All callers must migrate to async versions first
- Need ESLint rule to prevent new sync usage
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
go:needs-researchNeeds investigationNeeds investigationsquadSquad triage inbox — Lead will assign to a memberSquad triage inbox — Lead will assign to a membersquad:flightAssigned to Flight (Lead)Assigned to Flight (Lead)