Skip to content

Remove BaseSnippets class - #4839

Open
alejoe91 wants to merge 9 commits into
SpikeInterface:mainfrom
alejoe91:destroy-snippets
Open

alejoe91 wants to merge 9 commits into
SpikeInterface:mainfrom
alejoe91:destroy-snippets

Conversation

@alejoe91

@alejoe91 alejoe91 commented Oct 7, 2026

Copy link
Copy Markdown
Member

The BaseSnippets is unused and over-complicates the code base. Since it's not a common use case anymore (and it should not be) let's remove that concept.

@alejoe91 alejoe91 added this to the 0.106.0 milestone Oct 7, 2026
@alejoe91 alejoe91 added the core Changes to core module label Oct 7, 2026
@alejoe91
alejoe91 marked this pull request as ready for review October 14, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Changes to core module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants