Closed lambdalisue closed 10 months ago
The modifications in autoload/denops/plugin.vim
involve the removal of the s:DenopsPluginPost
function and an update to an autocommand that previously called s:relay_autocmd('DenopsPluginPost')
. Now, it directly invokes s:DenopsSystemPluginPost()
with the ++nested
flag. This appears to streamline event handling within the Denops plugin system for Vim/Neovim.
File | Change Summary |
---|---|
autoload/denops/plugin.vim |
Removed s:DenopsPluginPost function; updated DenopsSystemPluginPost autocommand to call s:DenopsSystemPluginPost() directly with ++nested . |
Objective | Addressed | Explanation |
---|---|---|
denops#plugin#is_loaded() should return 1 when called from DenopsPluginPost event (#311) |
❓ | The changes indicate an update in the event handling mechanism, which could affect the timing when denops#plugin#is_loaded() is called. However, without explicit mention that the change fixes the issue of the function returning 0, it's unclear if the issue has been directly addressed. |
In the code's burrow, deep and vast, A tiny fix was made at last. Events now nested, flow with grace, A rabbit's touch, in the digital space. 🐇💻
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
8e580f0
) 96.47% compared to head (089751f
) 96.47%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Close #311
Summary by CodeRabbit