Skip to content

Conversation

@FrancescoNegri
Copy link
Contributor

I had to close #26, therefore I opened a new pull request here.

@FrancescoNegri FrancescoNegri marked this pull request as ready for review March 13, 2025 17:50
@FrancescoNegri
Copy link
Contributor Author

@alejoe91 would you mind accepting the pull request if everything is in order? :)

@alejoe91
Copy link
Member

@FrancescoNegri can you add the annotation read/write to the tests to make sure it's working properly?

@FrancescoNegri
Copy link
Contributor Author

Sure!! I can do that tomorrow! 👍

@FrancescoNegri
Copy link
Contributor Author

@alejoe91 I added the tests. Please check what I did and let me know if it is correct, as I am quite new at unit testing!

@alejoe91 alejoe91 merged commit 0685e96 into SpikeInterface:main Mar 20, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants