-
Type: Task
-
Resolution: Done
-
Priority: Unknown
-
Affects Version/s: None
-
Component/s: None
Use Case
We want to drop support for snappy 6 and remove the logic that handles working across either major version. To help signal this change we should add snappy as an optional peerDep, with the version set to 7 or later
User Impact
- Users may have to upgrade their version of snappy
Dependencies
- Snappy
Acceptance Criteria
Implementation Requirements
- Remove version parsing logic for snappy
- Remove logic that handles promise or callback style API
- Add optional peer dep requirement for snappy version
- Import snappy lazily
Testing Requirements
- test lazy importing
- remove snappy version tests
Documentation Requirements
- our current docs for how to get started with snappy are still correct. Double check this incase any salient info is discovered during ticket implementation
- Migration guide note - none for existing snappy 7+ users, upgrade for others, none for nonsnappy users
- is related to
-
NODE-4522 Remove callbacks from all public method signatures
- Closed