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(documents): fix maxFileSize when it is loaded from g:coc_user_config #3610

Merged
merged 1 commit into from
Feb 1, 2022

Conversation

dmitmel
Copy link
Contributor

@dmitmel dmitmel commented Jan 31, 2022

It was broken by the commit a4e83d1.

@codecov
Copy link

codecov bot commented Jan 31, 2022

Codecov Report

Merging #3610 (e9ff84d) into master (c1a97aa) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3610      +/-   ##
==========================================
- Coverage   76.59%   76.55%   -0.05%     
==========================================
  Files         202      202              
  Lines       19977    19977              
  Branches     4446     4446              
==========================================
- Hits        15301    15293       -8     
- Misses       2925     2930       +5     
- Partials     1751     1754       +3     
Impacted Files Coverage Δ
src/core/documents.ts 88.65% <100.00%> (-0.62%) ⬇️
src/model/status.ts 88.09% <0.00%> (-4.77%) ⬇️
src/extensions.ts 57.92% <0.00%> (-0.50%) ⬇️
src/completion/index.ts 76.20% <0.00%> (-0.25%) ⬇️
src/services.ts 22.75% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c1a97aa...e9ff84d. Read the comment docs.

@chemzqm chemzqm merged commit 2b6a13d into neoclide:master Feb 1, 2022
chemzqm added a commit that referenced this pull request Feb 1, 2022
2b6a13d fix(documents): fix maxFileSize when it is loaded from coc_user_config (#3610)
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