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

Make it clear that popup delay depends on updatetime option #22

Closed
metalelf0 opened this issue Mar 9, 2021 · 2 comments
Closed

Make it clear that popup delay depends on updatetime option #22

metalelf0 opened this issue Mar 9, 2021 · 2 comments
Labels
bug Something isn't working released

Comments

@metalelf0
Copy link

Describe the bug

I'm not entirely sure this is related to vim-ultest rather than nvim itself, but when I first installed the plugin, the popup window with the failed test information took exactly 4 seconds to display (the one that pops up on hover on the first line of the failed test).

I checked with :options and found out that I had updatetime set to 4000 (which, I guess, is the default value). Reducing it to 100 actually makes the popup window appear instantly.

However, this will cause a lot more writes (40 times more) to the swap file, if it's used, therefor raising a potential issue on SSD devices. I would add this information to the readme, either telling users to disable swap files or to know the tradeoffs of a low updatetime.

Thanks for this plugin and keep up the great work!

@metalelf0 metalelf0 added the bug Something isn't working label Mar 9, 2021
@rcarriga
Copy link
Owner

rcarriga commented Mar 9, 2021

It shouldn't depend on the updatetime value, I can confirm I set mine to 4000 and it still opens immediately. I imagine there is something else causing the delay. Could you provide some more info on your version, system and logs please?

Edit: On a second look I see I was wrong, it does depend on updatetime. I will add this to the docs, thanks!

@github-actions
Copy link

🎉 This issue has been resolved in version 2.5.8 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released
Projects
None yet
Development

No branches or pull requests

2 participants