aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | | Increase default LSP sync timeout to 1000msKarim Abou Zeid2021-05-02
| | | | | | | |
| * | | | | | | Add client.request_sync docKarim Abou Zeid2021-05-02
| | | | | | | |
| * | | | | | | Synchronous formatting methods notify the user on timeout and interruptedKarim Abou Zeid2021-05-02
| | | | | | | |
| * | | | | | | Support multiple range formatting clientsKarim Abou Zeid2021-05-02
| | | | | | | |
| * | | | | | | doc clarificationKarim Abou Zeid2021-05-02
| | | | | | | |
| * | | | | | | Add formatting_seq_sync, change formatting and formatting_syncKarim Abou Zeid2021-05-01
| | | | | | | |
* | | | | | | | signs: fix overflow during adjustment on Windows (#14472)statiolake2021-05-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On Windows, `new_lnum + MAXLNUM` causes overflow and as a result the line number of that sign becomes invalid negative number. This occurs when the `set signcolumn=yes`, in other words `signcolumn` is not `auto` and the sign column is less than 2 columns. The related change was made in the commit f2ed7605da45eb79a4f7bb89fb19f680fb5a4927. Originally the above addition is only executed if `amount != MAXLNUM`, so reintroducing this check fixes the bug and will hardly produces a new bug. Fixes https://github.com/neovim/neovim/issues/14460
* | | | | | | | lsp: fix regression in jump_to_location() (#14446)francisco souza2021-05-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a regression that was introduced in [1] by accidentally leaving out a line. 1: https://github.com/neovim/neovim/pull/12262
* | | | | | | | fix: make nvim_win_get_number() return win number (#14470)Julian Berman2021-05-01
| | | | | | | | | | | | | | | | | | | | | | | | Closes https://github.com/neovim/neovim/issues/14467
* | | | | | | | docs: Treesitter (#13260)TJ DeVries2021-05-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * doc & fixes: Generate treesitter docs * fixup to treesitter-core * docs(treesitter): fix docs for most functions Co-authored-by: Thomas Vigouroux <tomvig38@gmail.com>
* | | | | | | | doc: update vim.cmd() (#14450)Shadman2021-05-01
| | | | | | | |
* | | | | | | | Merge pull request #14380 from bfredl/virtcolBjörn Linse2021-05-01
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | decorations: right_align and win_col
| * | | | | | | | decorations: right_align and win_colBjörn Linse2021-05-01
| |/ / / / / / /
* | | | | | | | Merge pull request #14463 from mfussenegger/decoded-result-falseMichael Lingelbach2021-04-30
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | lsp: Allow decoded.result to be `false`
| * | | | | | | | lsp: Allow decoded.result to be `false`Mathias Fussenegger2021-04-30
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some servers might respond to `workspace/executeCommand` requests with a boolean result and that could be `false`. A `false` result should be allowed and not trigger the `on_error` handler: -- Invalid server message on_error(client_errors.INVALID_SERVER_MESSAGE, decoded) Concrete example where this occurred is with eclipse.jdt.ls: vim.lsp.buf_request( 0, 'workspace/executeCommand', { command = 'java.project.isTestFile', arguments = { vim.uri_from_bufnr(0), }, }, function(err, _, resp) print(vim.inspect(err), vim.inspect(resp)) end )
* | | | | | | | Merge pull request #14361 from bfredl/synenableBjörn Linse2021-04-30
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | [WIP] startup: do "syntax enable" instead of "syntax on" after init.vim
| * | | | | | | startup: do "syntax enable" instead of "syntax on"Björn Linse2021-04-30
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "syntax on" overwrites existing highlight groups, while "syntax enable" just sets missing groups. This change prevents user defined highlights in init.vim/lua to get overwritten. The manual recommends "syntax enable" for new configurations anyway, "on" command was probably used as it is the implicit default.
* | | | | | | Merge pull request #14459 from jamessan/pvs-scriptJames McCoy2021-04-29
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | fix(pvscheck): Switch URL to pvs-studio.com
| * | | | | | fix(pvscheck): Switch URL to pvs-studio.comJames McCoy2021-04-29
|/ / / / / / | | | | | | | | | | | | | | | | | | [skip ci]
* | | | | | Merge pull request #14448 from jamessan/nightly-ci-permissionsJames McCoy2021-04-27
|\ \ \ \ \ \ | | | | | | | | | | | | | | Update permissions for vim-patches workflow
| * | | | | | ci(gha/vim-patches): Create PR against branch running the workflowJames McCoy2021-04-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoids hard-coding branch names. [skip ci]
| * | | | | | ci(gha): Rename nightly workflow to vim-patchesJames McCoy2021-04-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [skip ci]
| * | | | | | ci(gha/nightly): Enable "contents: write" permissions to setup PRJames McCoy2021-04-26
|/ / / / / / | | | | | | | | | | | | | | | | | | [skip ci]
* | | | | | Merge pull request #14416 from bfredl/floatorderBjörn Linse2021-04-26
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | ui_compositior: handle overlapping displayed floats in the same tick
| * | | | | ui_compositior: handle multiple displayed floats in the same tickBjörn Linse2021-04-26
|/ / / / / | | | | | | | | | | | | | | | | | | | | problem: the order of non-focuesed float opened before focused float is wrong (sunjon) solution: check curwin and correct the order (bfredl)
* | | | | Merge pull request #14435 from michaelb/doc_missing_nvim_open_termThomas Vigouroux2021-04-24
|\ \ \ \ \ | | | | | | | | | | | | doc : fixing missing @return in nvim_open_term doc
| * | | | | doc : fixing missing @return in nvim_open_term docMichael Bleuez2021-04-24
|/ / / / /
* | | | | Merge pull request #14429 from ckipp01/forceMichael Lingelbach2021-04-23
|\ \ \ \ \ | | | | | | | | | | | | [LSP] - Don't automatically force shutdown on second restart.
| * | | | | Don't automatically force shutdown on second restart.ckipp012021-04-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is maybe a bit of a niche case, but I hit on this often as I'm developing a server, and therefore continually restarting it to get the latest changes of the server. Previously, I could only do this once since if you send in a request to restart/shut down the server, it will register it as a `tried_graceful_shutdown = true` meaning that the next restart would force it to be killed instead of another graceful exit. Instead, this changes the name a bit and now it will only mark `graceful_shutdown_failed = true` _if_ it actually fails to gracefully shutdown. This change allows for a user to restart multiple times in a situation like mine where nothing is going wrong, but I just want to restart continually as I'm developing without having to close and reopen.
* | | | | | lsp: make tagstack smarter motion-wise (#12262)Gabriel Sanches2021-04-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit prevents two things regarding the tagstack and jumping to locations: - Pushing the same item twice in a row - Pushing an item where the destination is the same as the source Both prevent having to press CTRL-T additional times just to pop items that don't make the cursor move.
* | | | | | Merge pull request #14432 from mjlbach/revert_14428Michael Lingelbach2021-04-23
|\ \ \ \ \ \ | | | | | | | | | | | | | | Revert "lsp: fix blocking in closing of clients"
| * | | | | | Revert "lsp: fix blocking in closing of clients"Michael Lingelbach2021-04-23
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 2e6c09838f88803f31d229002715628639631897. * Fixes #14428 * This commit caused neovim to close while open handles to the uv timer to kill active language servers were still open
* | | | | | lsp: check if config is nilMarco Hinz2021-04-23
| | | | | |
* | | | | | lsp: sort diagnostics by severity (#14372)Marco Hinz2021-04-23
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow to sort diagnostics (and thus signs and virtual text) by severity, so that the most important message is shown first. vim.lsp.handlers['textDocument/publishDiagnostics'] = vim.lsp.with( vim.lsp.diagnostic.on_publish_diagnostics, { severity_sort = true, } ) Fixes https://github.com/neovim/neovim/issues/13929
* | | | | vim-patch:8.2.2776: :mksession uses current value of 'splitbelow' and ↵Jan Edmund Lazo2021-04-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'splitright' (#14398) Problem: :mksession uses current value of 'splitbelow' and 'splitright' even though "options" is not in 'sessionoptions'. (Maxim Kim) Solution: Save and restore the values, instead of setting to the current value. (closes vim/vim#8119) https://github.com/vim/vim/commit/0995c81f2ffe276669daa004f7778ecc6f5ee09d Patch v8.2.1682 is not ported. Replace "goto fail;" with "return FAIL;".
* | | | | lua: make vim.cmd an alias of vim.api.nvim_exec() (#14401)Shadman2021-04-22
| | | | | | | | | | | | | | | | | | | | | | | | | Previously vim.cmd was an alias of nvim_command(). From now on it is an alias of nvim_exec().
* | | | | Merge pull request #14411 from seandewar/vim-8.2.1588Jan Edmund Lazo2021-04-21
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:8.2.1588 - port `prompt_getprompt()`
| * | | | | fix(oldtest): fix assert_match() for :buffers tSean Dewar2021-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previous pattern could cause test to fail if ":buffers t" reported "1 second ago".
| * | | | | vim-patch:8.2.1588: cannot read back the prompt of a prompt bufferSean Dewar2021-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot read back the prompt of a prompt buffer. Solution: Add prompt_getprompt(). (Ben Jackson, closes vim/vim#6851) https://github.com/vim/vim/commit/077cc7aa0e0c431e97795612374fe17fe7c88803 Updated prompt_getprompt() doc to https://github.com/vim/vim/commit/cb80aa2d53e56d3aba3b3c439fb467f29a750c5e and removed mention of method syntax usage (not supported by Nvim).
| * | | | | oldtest: port test_prompt_buffer.vimSean Dewar2021-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Included from before v8.2.1588. Required for v8.2.1588. Many tests use stuff like term_sendkeys() which is N/A, so are skipped.
| * | | | | eval: port tv_get_buf_from_arg()Sean Dewar2021-04-21
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cherry-picked from v8.2.1562, but uses Nvim's tv_check_str_or_nr(). Required for v8.2.1588. It isn't used for f_bufnr() to avoid a double error message if the first argument isn't a number or string similiar to what's seen in Vim.
* | | | | Merge pull request #14417 from jamessan/ci-updateMarco Hinz2021-04-21
|\ \ \ \ \ | | | | | | | | | | | | GHA updates
| * | | | | ci(gha/release): Enable "contents: write" permission to publish releaseJames McCoy2021-04-21
| | | | | |
| * | | | | ci(gha/win): Specify valid directory for DEPS_BUILD_DIRJames McCoy2021-04-21
| | | | | |
| * | | | | ci(gha): Simplify clang installationJames McCoy2021-04-21
| | | | | |
| * | | | | ci(gha): Update to clang-12James McCoy2021-04-21
| | | | | |
* | | | | | api: fix nvim_exec() silencing behaviour (#14413)Shadman2021-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously nvim_exec would silent output no matter whether output is true or false. Now output is only silent and captured when output is true.
* | | | | | gitignore: add vim patches (#14412)Shadman2021-04-21
|/ / / / /
* | | | | win/package: update wintools.zip (#14408)Jan Edmund Lazo2021-04-20
| | | | | | | | | | | | | | | | | | | | | | | | | This does not have 7za.exe and tidy.exe. Ref: https://github.com/neovim/neovim/issues/14078
* | | | | win/package: remove cat,tidy (#14402)Jan Edmund Lazo2021-04-20
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Neovim should not bundle external tools that are not needed in the runtime environment. cat.exe is meant for tests only. Install a mingw/msys2/busybox environment which bundle cat.exe. tidy.exe was never used in tests and is not required in Neovim runtime. busybox and tidy.exe can be installed via scoop. Ref: https://github.com/neovim/neovim/issues/14078