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

fix: Use @denops/std/helper/getbufinfo instead #4

Merged
merged 2 commits into from
Feb 23, 2025

Conversation

mityu
Copy link
Contributor

@mityu mityu commented Feb 21, 2025

Related to: vim-denops/deno-denops-std#279

As you know, the getbufinfo function in @denops/std/function will raise an error when there's a funcref in buffer-local variables. So, we should switch to the alternative getbufinfo function provided by @denops/std/helper/getbufinfo module.

I searched for "getbufinfo" by rg getbufinfo at the root of this repository, and changed all of them to the helper module's getbufinfo function.

Copy link

codecov bot commented Feb 22, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 50.65%. Comparing base (a21bd0a) to head (bd568d2).

Files with missing lines Patch % Lines
builtin/source/buffer.ts 50.00% 1 Missing ⚠️
builtin/source/line.ts 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   50.63%   50.65%   +0.02%     
==========================================
  Files          77       77              
  Lines        2277     2278       +1     
  Branches       67       67              
==========================================
+ Hits         1153     1154       +1     
  Misses       1124     1124              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

The getbufinfo function in @denops/std/function will raise an error when
there's a funcref in buffer-local variables, and we should switch to the
alternative getbufinfo function provided by @denops/std/helper/getbufinfo.
@mityu mityu force-pushed the use-helper-getbufinfo branch from 48c529c to bd568d2 Compare February 22, 2025 15:27
@mityu
Copy link
Contributor Author

mityu commented Feb 22, 2025

Fixed lint errors and rebased the main branch.

@lambdalisue lambdalisue merged commit 16197bc into vim-fall:main Feb 23, 2025
6 of 7 checks passed
@mityu mityu deleted the use-helper-getbufinfo branch February 23, 2025 01:06
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