Closed Shougo closed 2 months ago
The recent update enhances the Denops plugin system by using a registered plugin list instead of discovering plugins dynamically. This change ensures that denops#cache#update()
works correctly for plugins that do not use main.ts
, improving reliability and consistency in plugin management.
Files | Change Summary |
---|---|
autoload/denops/cache.vim |
Replaced denops#_internal#plugin#collect() with denops#plugin#registered_files() |
autoload/denops/plugin.vim |
- Added s:registered_files dictionary- Updated denops#plugin#load to register files- Added denops#plugin#registered_files function |
sequenceDiagram
participant User
participant DenopsPlugin
participant Cache
User->>DenopsPlugin: Call `denops#plugin#load`
DenopsPlugin->>DenopsPlugin: Register files in `s:registered_files`
User->>Cache: Call `denops#cache#update()`
Cache->>DenopsPlugin: Call `denops#plugin#registered_files()`
DenopsPlugin->>Cache: Return registered files
Cache->>Cache: Update cache with registered files
Objective | Addressed | Explanation |
---|---|---|
Use registered plugins instead of discover in denops#cache#update() (#360) |
✅ |
In the land of code where plugins play,
A change has come to light the way.
No more searching, files in hand,
Denops now a better land.
Cache updated, smooth and bright,
Plugins load with pure delight.
🐇✨
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:
Project coverage is 90.23%. Comparing base (
6e14ad1
) to head (4c2fd1b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I'll apply some changes.
Closed in favor of https://github.com/vim-denops/denops.vim/pull/367
OK.
Fix #360
Summary by CodeRabbit
New Features
Improvements