Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update the quickfix help with more examples #2427

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Update the quickfix help with more examples #2427

wants to merge 1 commit into from

Conversation

@yegappan
Copy link
Member

@yegappan yegappan commented Dec 8, 2017

  1. Update the quickfix help with more examples
  2. When the quickfix window is not present, return zero for getqflist() with 'winid'.
  3. Add more tests for jumping to quickfix list entries.

@yegappan
Update the doc with examples. Add more tests for jumping to quickfix …
da8e9a9 
…entries. Return zero if the quickfix window is not present
@codecov-io
Copy link

Codecov Report

Merging #2427 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2427      +/-   ##
==========================================
+ Coverage   74.44%   74.49%   +0.05%     
==========================================
  Files          92       92              
  Lines      132705   132708       +3     
  Branches    29111    31007    +1896     
==========================================
+ Hits        98789    98858      +69     
+ Misses      33881    33819      -62     
+ Partials       35       31       -4
Impacted Files Coverage Δ
src/quickfix.c 92.62% <100%> (+0.12%) ⬆️
src/channel.c 82.44% <0%> (-0.05%) ⬇️
src/gui_gtk_x11.c 47.7% <0%> (ø) ⬆️
src/screen.c 74% <0%> (+0.02%) ⬆️
src/ex_cmds.c 77.94% <0%> (+0.08%) ⬆️
src/if_py_both.h 76.59% <0%> (+0.14%) ⬆️
src/gui.c 47.45% <0%> (+0.15%) ⬆️
src/term.c 50.72% <0%> (+0.16%) ⬆️
src/message.c 68.72% <0%> (+0.24%) ⬆️
... and 4 more

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 e4b78e2...da8e9a9. Read the comment docs.

@brammool brammool closed this in 74240d3 Dec 10, 2017
adizero pushed a commit to adizero/vim that referenced this pull request May 19, 2018
@brammool @adizero
patch 8.0.1384: not enough quickfix help; confusing winid
c98ad9e 
Problem:    Not enough quickfix help; confusing winid.
Solution:   Add more examples in the help. When the quickfix window is not
            present, return zero for getqflist() with 'winid'. Add more tests
            for jumping to quickfix list entries. (Yegappan Lakshmanan, closes
            vim#2427)
janlazo added a commit to janlazo/neovim that referenced this pull request Jun 6, 2019
@janlazo
vim-patch:8.0.1384: not enough quickfix help; confusing winid
f233ced 
Problem:    Not enough quickfix help; confusing winid.
Solution:   Add more examples in the help. When the quickfix window is not
            present, return zero for getqflist() with 'winid'. Add more tests
            for jumping to quickfix list entries. (Yegappan Lakshmanan, closes
            vim/vim#2427)
vim/vim@74240d3
janlazo added a commit to janlazo/neovim that referenced this pull request Jun 7, 2019
@janlazo
vim-patch:8.0.1384: not enough quickfix help; confusing winid
773c488 
Problem:    Not enough quickfix help; confusing winid.
Solution:   Add more examples in the help. When the quickfix window is not
            present, return zero for getqflist() with 'winid'. Add more tests
            for jumping to quickfix list entries. (Yegappan Lakshmanan, closes
            vim/vim#2427)
vim/vim@74240d3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yegappan @codecov-io