aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #14366 from mfussenegger/text-document-version-zeroMichael Lingelbach2021-04-14
|\ | | | | lsp: Accept text document edits with version zero
| * lsp: Remove vim.NIL handling from apply_text_document_editMathias Fussenegger2021-04-14
| | | | | | | | | | The rpc layer normalizes `vim.NIL` to `nil`, so the scenario tested should never happen.
| * lsp: Accept text document edits with version zeroMathias Fussenegger2021-04-14
|/ | | | | | | | | | | | | | | | | | | | | There were a couple of reports of "Buffer X newer than edits" problems. We first assumed that it is incorrect for a server to send 0 as a version - and stated that they should send a `null` instead, given that in the specification the `textDocument` of a `TextDocumentEdit` is a `OptionalVersionedTextDocumentIdentifier`. But it turns out that this was a change in 3.16, and in 3.15 and earlier versions of the specification it was a `VersionedTextDocumentIdentifier` and language servers didn't have a better option than sending `0` if they don't keep track of the version numbers. So this changes the version check to always accept `0` values. See - https://github.com/neovim/neovim/issues/12970 - https://github.com/neovim/neovim/issues/14256 - https://github.com/haskell/haskell-language-server/pull/1727
* doc: prefer "python -m pip" (#14353)Aru Sahni2021-04-14
| | | | | | | | | | | | | | | | | | | | The current guidance for install Python packages is to use python -m pip install <package_name> Instead of pip install <package_name> This ensures that one is using the version of pip that is tied to the environment's interpreter (and, thusly, its packages). This has [been endorsed by a core maintainer](https://snarky.ca/why-you-should-use-python-m-pip/) as being the recommended way to invoke pip. As there currently are a few places where the old invocation was used, attempt to bring them in line. Fixes #14234
* doc: port prompt-buffer section (#14342)Sean Dewar2021-04-14
| | | | | | | | | | | [skip ci] Changes from original include: - "See |terminal-window|" -> "See |terminal|". - Remove mention of using CTRL-W window commands in insert mode. - Converted usage example to use the Nvim job and channel API. - Removed logging from usage example, as ch_logfile() has no direct Nvim counterpart. - Fixed some small grammar/spelling mistakes.
* lsp: fix _make_floating_popup_size when wrap_at is nil (#14359)Michael Lingelbach2021-04-14
|
* Merge pull request #14046 from ↵Thomas Vigouroux2021-04-14
|\ | | | | | | | | nvim-treesitter/feature/language-tree-directive-config feat(treesitter): allow injections to be configured through directives
| * feat(treesitter): allow injections to be configured through directivesSteven Sojka2021-04-02
| |
* | lsp: fix off-by-one in line diagnostic highlightingMarco Hinz2021-04-14
| |
* | Merge pull request #14119 from mfussenegger/lsp-debounceMichael Lingelbach2021-04-13
|\ \ | | | | | | lsp: Add a flag to debounce didChange notifications
| * | lsp: Add a flag to debounce didChange notificationsMathias Fussenegger2021-04-13
| | | | | | | | | | | | Would help with cases as reported in https://github.com/neovim/neovim/issues/14087
* | | Merge pull request #14324 from mhinz/lsp-border-float-placementMarco Hinz2021-04-13
|\ \ \ | |/ / |/| |
| * | lsp: correct float placement when using bordersMarco Hinz2021-04-12
| | | | | | | | | | | | | | | Because borders add up to 2 to the height of a float, we need to subtract that from the anchor position, when opening a float in the lower half of the window.
| * | lsp: make sure borders are always inside of the screenMarco Hinz2021-04-12
| | | | | | | | | | | | | | | Add a helper function to get the additional width a border adds to a float to make sure that the border is always inside the visible screen.
* | | Merge pull request #14318 from chentau/extmark_luadoBjörn Linse2021-04-13
|\ \ \ | | | | | | | | extmark: splice extmarks on :luado
| * | | extmark: splice extmarks on :luadochentau2021-04-12
| | | |
* | | | tutor: force one sign per line (#14352)Marco Hinz2021-04-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only thing setting signs in the tutor buffer (buftype=nofile), should be tutor.vim itself. Instead of endlessly piling up new signs per line, just unplace any sign on the line before placing a new one. Closes https://github.com/neovim/neovim/issues/13808 Closes https://github.com/neovim/neovim/pull/13809
* | | | float: add "solid" border style (#14310)Marco Hinz2021-04-12
| |/ / |/| | | | | | | | | | | | | | It looks solid with the default `FloatBorder` group. If you set the bgcolor of FloatBorder to the same color as for FloatNormal, you effectively get an "1-cell padding".
* | | Merge pull request #14337 from janlazo/vim-8.2.0409Jan Edmund Lazo2021-04-11
|\ \ \ | | | | | | | | vim-patch:8.1.{1841,2416},8.2.{409,1920,2050}
| * | | test/old: ':execute' does not suppress range errorJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | :2,1> should trigger backwards range error but it is suppressed for inccomand. :execute "2,1>" does not suppress the error.
| * | | vim-patch:8.1.2416: loading menus sets v:errmsgJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Loading menus sets v:errmsg. Solution: Avoid setting v:errmsg and add a test for that. (Jason Franklin) https://github.com/vim/vim/commit/e24c5b3332b453175e5f73423884087a4aef1247
| * | | vim-patch:8.1.1841: no test for Ex shift commandsJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for Ex shift commands. Solution: Add a test. (Dominique Pelle, closes vim/vim#4801) https://github.com/vim/vim/commit/d5e3cc11d391f3aa80bf66f91001f11a3a145b43 Comment out some tests because of Neovim inccomand.
| * | | vim-patch:8.2.1920: listlbr test fails when run after another testJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Listlbr test fails when run after another test. Solution: Add test separately to list of test targets. https://github.com/vim/vim/commit/cf4d454df0619ee41ef40e7e91fce3fb061d7d5b
| * | | vim-patch:8.2.2050: search test contains unneeded sleepsJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Search test contains unneeded sleeps. Solution: Rename the function, remove sleeps. (Christian Brabandt, closes vim/vim#7369) https://github.com/vim/vim/commit/6bed0dbc8500be3ea751cc527a6ee89ca073a4d1
| * | | vim-patch:8.2.0409: search test leaves file behindJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Search test leaves file behind. Solution: Delete the file. Also use Check commands. https://github.com/vim/vim/commit/b68df220c5278365792b8f94d8d6f34d329b7746 Reorder tests based on test_search.vim in patch v8.2.0409.
* | | | Merge pull request #14343 from jamessan/doc-fixJames McCoy2021-04-11
|\ \ \ \ | |/ / / |/| | | fix(doc): Add '/site' to stdpath('data') example in `:help 'rtp'`
| * | | fix(doc): Add '/site' to stdpath('data') example in `:help 'rtp'`James McCoy2021-04-10
| | | | | | | | | | | | | | | | [skip ci]
* | | | Merge pull request #14339 from janlazo/vim-8.2.0044Jan Edmund Lazo2021-04-11
|\ \ \ \ | | | | | | | | | | vim-patch:8.2.{44,45,69,2319}
| * | | | vim-patch:8.2.2319: "exptype_T" can be read as "expected type"Jan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: "exptype_T" can be read as "expected type". Solution: Rename to "exprtype_T", expression type. https://github.com/vim/vim/commit/657137ca487c60d63989236115115161def270a5 Vim9 changes omitted.
| * | | | vim-patch:8.2.0069: ETYPE_ is used for two different enumsJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ETYPE_ is used for two different enums. Solution: Rename one to use EXPR_. https://github.com/vim/vim/commit/87396072c5c468f0c129e4ec7cd944ac897b7752
| * | | | vim-patch:8.2.0045: script test failsJan Edmund Lazo2021-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Script test fails. Solution: For numbers "is" and "isnot" work like "==" and "!=". https://github.com/vim/vim/commit/ec57ec692eb228ee061824a190d7c451f029c430
| * | | | vim-patch:8.2.0044: expression type is used inconsistentlyJan Edmund Lazo2021-04-11
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Expression type is used inconsistently. Solution: Add "ETYPE_IS" and "ETYPE_ISNOT" as separate enum values. Rename "TYPE_" to "ETYPE_" to avoid confusion. https://github.com/vim/vim/commit/07a3db89b8953bd0964895badb3b662f7514bc10
* | | | Merge pull request #14340 from mjlbach/fix_clangd_nixMichael Lingelbach2021-04-10
|\ \ \ \ | |/ / / |/| | | contrib: add clang-tools to nix flake
| * | | contrib: add clang-tools to nix flakeMichael Lingelbach2021-04-10
| | | |
* | | | test/ui/cmdline: set notimeout to remove indeterminism (#14338)Jan Edmund Lazo2021-04-10
| | | |
* | | | Merge pull request #14317 from chentau/extmark_subBjörn Linse2021-04-10
|\ \ \ \ | |/ / / |/| | | extmark: correct extmark_splice call with substitute and inccommand when replacing with escaped backslashes
| * | | extmark: correct extmark_splice call with inccommandchentau2021-04-09
| |/ /
* | | doc: advertise nanotee/nvim-lua-guide (#14332) [skip ci]Christian Clason2021-04-09
| | | | | | | | | | | | | | | Mention https://github.com/nanotee/nvim-lua-guide at the beginning of ":h lua" as well as ":h lua-vimscript". Closes #12369 .
* | | Merge pull request #14321 from jamessan/ci-fixesJames McCoy2021-04-09
|\ \ \ | | | | | | | | Some small cleanups/diagnostic improvements
| * | | fix(test): Detect more core filenamesJames McCoy2021-04-08
| | | |
| * | | Add more info to debug messagesJames McCoy2021-04-08
| | | | | | | | | | | | | | | | [skip ci]
| * | | Lower "closed by the client" message level to INFOJames McCoy2021-04-08
| | | | | | | | | | | | | | | | [skip ci]
* | | | Merge pull request #14326 from janlazo/vim-8.2.0623Jan Edmund Lazo2021-04-09
|\ \ \ \ | | | | | | | | | | vim-patch:8.2.{623,1822,2738}
| * | | | vim-patch:8.2.2738: extending a list with itself can give wrong resultJan Edmund Lazo2021-04-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Extending a list with itself can give wrong result. Solution: Remember the item before where the insertion happens and skip to after the already inserted items. (closes vim/vim#1112) https://github.com/vim/vim/commit/dcae51facc4d6de1edd62f0242b40972be841103 Originated from Neovim commit 7ceebacb3fad49ba8321397cf839948caa55b3f5.
| * | | | vim-patch:8.2.1822: list test doesn't failJan Edmund Lazo2021-04-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: List test doesn't fail. Solution: Adjust the test for NULL list handling. https://github.com/vim/vim/commit/f57497276bc616d3856eeff9824c080941faa51b Comment out test cases that modify null lists because Neovim throws error messages instead of silently failing. Null lists should be read-only and constant. https://github.com/neovim/neovim/issues/4615
| * | | | eval: add v:_null_stringJan Edmund Lazo2021-04-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replacement for Vim's test_null_string(). Vim uses it to verify that its codebase handles null strings. Preparation for the Test_null_list() in patch v8.2.1822. Use v:_null_string, not non-existent env var, for null string tests. Mention v:_null_string in id() because id(v:_null_string) returns (nil).
| * | | | vim-patch:8.2.0623: typo in test commentJan Edmund Lazo2021-04-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Typo in test comment. (Christ van Willegen) Solution: Avoid mixing up a data structure with a body part. https://github.com/vim/vim/commit/f7b398c6a9476a2004a42555b731ebf47b866408 Cherry-pick Test_dict_lock_operator() from patch v8.2.0619.
* | | | | vim-patch:8.2.2737: status line not updated when local 'statusline' option ↵bphilly962021-04-08
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | set (#14325) Problem: Status line not updated when local 'statusline' option set. Solution: Check the 'statusline' option of each window. https://github.com/vim/vim/commit/d8db8383926cb8729417d9515cbfaf455dbbd8d1
* | | | Merge pull request #14314 from janlazo/vim-8.2.2720Jan Edmund Lazo2021-04-08
|\ \ \ \ | |/ / / |/| | | vim-patch:8.0.{1505,1511},8.2.{1304,1305,1393,1508,1791,2720}
| * | | vim-patch:8.2.1305: some tests are still old styleJan Edmund Lazo2021-04-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Some tests are still old style. Solution: Convert tests 52 and 70 to new style. (Yegappan Lakshmanan, closes vim/vim#6544) Fix error in FinishTesting(). https://github.com/vim/vim/commit/622b3568fa1baf07671d31390815fb0a55a99891 Omit test_mzscheme.vim. Neovim does not support "mzscheme" feature.