Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(plugins): neotest integration #48

Merged
merged 1 commit into from
Dec 10, 2023
Merged

feat(plugins): neotest integration #48

merged 1 commit into from
Dec 10, 2023

Conversation

igorlfs
Copy link
Contributor

@igorlfs igorlfs commented Dec 9, 2023

Hello!

This is just a simple integration with neotest. More specifically, it's summary. Currently, when the plugin tries to restore neotest's summary, an error is thrown. The integration fixes this issue, by using a workaround similar to dapui's: first calling close() and then calling open(). Neotest also provides others "views", such as the output panel, but I don't use them, so I can't say if they're causing trouble as of now, but I can take a look if you don't wanna merge an "incomplete" PR

@jedrzejboczar
Copy link
Owner

Thanks! I'm not using neotest myself but this looks good to me. I think we can merge only with the "summary" view and possible improve later if needed.

@jedrzejboczar jedrzejboczar merged commit 4665cee into jedrzejboczar:master Dec 10, 2023
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