aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | | feat: aggregate code actions from all clients (#15121)Folke Lemaitre2021-07-18
|/ / / / /
* | | | | fix(ci): remove mongodb from brew to avoid brew upgrade error (#15115)Oliver Marriott2021-07-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | macOS CI was failing because: - brew upgrade fails because, - mongodb-community cant upgrade because, - some symlinks are owned by ... mongodb-community... We don't use mogodb, so we can just remove it wholesale.
* | | | | doc(lsp): various small fixes (#15113)Ido Ariel2021-07-17
|/ / / / | | | | | | | | | | | | | | | | | | | | - remove incorrect usage of docstrings - fix make_formatting_params return type documentation to 'DocumentFormattingParams' - make progress_handler private - fix links
* | | | Merge pull request #15078 from bfredl/p_vim_viBjörn Linse2021-07-14
|\ \ \ \ | | | | | | | | | | refactor(options): remove obsolete distinction of "vi" vs "vim" defaults
| * | | | refactor(options): remove obsolete distinction of "vi" vs "vim" defaultsBjörn Linse2021-07-14
| | | | | | | | | | | | | | | | | | | | | | | | | It might come as a schocking surprise, but the defaults we support are the NEOVIM defaults.
| * | | | refactor(options): delet unused P_VIM flagBjörn Linse2021-07-14
| |/ / /
* | | | Merge pull request #15017 from donbex/local-file-uriMichael Lingelbach2021-07-14
|\ \ \ \ | |/ / / |/| | | fix(lsp): accept file URIs without a hostname
| * | | feat(vim.uri): Allow URI schemes other than file: without authorityTom Payne2021-07-10
| | | |
| * | | test(lsp): file URIs do not allow relative pathsAlessandro Pezzoni2021-07-10
| | | |
| * | | test(lsp): file URIs without a hostnameAlessandro Pezzoni2021-07-10
| | | |
| * | | fix(lsp): accept file URIs without a hostnameAlessandro Pezzoni2021-07-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | RFC 8089, which defines the file URI scheme, also allows URIs without a hostname, i.e. of the form file:/path/to/file. These are returned by some language servers and accepted by other LSP implementations, such as VSCode's, so it is reasonable for us to accept them as well.
* | | | fix(lsp): pass bufnr for async formatting (#15084)Michael Lingelbach2021-07-14
| | | | | | | | | | | | | | | | | | | | | | | | the `textDocument/rangeFormatting` nad `textDocument/formatting` did not pass bufnr to apply_text_edits, meaning edits were applied to the user's currently active buffer. This could result in text being applied to the wrong buffer.
* | | | Merge pull request #14812 from gpanders/stdin_closedBjörn Linse2021-07-13
|\ \ \ \ | | | | | | | | | | feat(job): add parameter to close stdin
| * | | | Add test case for 'null' stdin modeGregory Anders2021-07-12
| | | | |
| * | | | Fix line length clint errorGregory Anders2021-07-12
| | | | |
| * | | | Rename stdin to stdin_modeGregory Anders2021-07-12
| | | | | | | | | | | | | | | | | | | | stdin is a macro in Windows builds.
| * | | | feat(job): add parameter to close stdinGregory Anders2021-07-12
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some programs behave differently when they detect that stdin is being piped. This can be problematic when these programs are used with the job control API where stdin is attached, but not typically used. It is possible to run the job using a PTY which circumvents this problem, but that includes a lot of overhead when simply closing the stdin pipe would suffice. To enable this behavior, add a new parameter to the jobstart options dict called "stdin" with two valid values: "pipe" (the default) implements the existing behavior of opening a channel for stdin and "null" which disconnects stdin (or, if you prefer, connects it to /dev/null). This is extensible so that other modes can be added in the future.
* | | | fix(lsp): avoid ipairs on non-sequential tables (#15059)Michael Lingelbach2021-07-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ipairs terminates on the first nil index when iterating over table keys: for i,k in ipairs( {[1] = 'test', [3] = 'test'} ) do print(i, k) end prints: 1 test Instead, use pairs which continues iterating over the entire table: for i,k in pairs( {[1] = 'test', [3] = 'test'} ) do print(i, k) end prints: 1 test 3 test
* | | | chore(clang): Fix 'Array subscript is undefined' #15057ansimita2021-07-11
| | | |
* | | | fixup(clipboard): Fix error not properly handled #14984Shadman2021-07-11
| | | | | | | | | | | | fixes #14967
* | | | fix(lsp): Ensure human readable errors are printedMathias Fussenegger2021-07-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `return err_message(tostring(err))` caused errors to be printed as `table: 0x123456789` instead of showing the error code and error message. This also removes some `if err` blocks that never got called because at the end of `handlers.lua` all the handlers are wrapped with logic that adds generic error handling.
* | | | fix(lsp): Ensure users get feedback on references/symbols errors or empty ↵Mathias Fussenegger2021-07-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | results Relates to https://github.com/neovim/neovim/issues/15050 Users should get some indication if there was an error or an empty result.
* | | | window.c: address Coverity failure #14996Daniel Steinberg2021-07-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check that `wip2` does not point to the same address as `wip`, to address the Coverity test failure from PR #14884. Based on the `if` clauses, `free_wininfo(wip2, ...)` is only called when `wip2->wi_win == NULL` and `wip->wi_win == wp`. I think `wip2` would only point to the same address as `wip` in scenarios where `wp` were `NULL`, which can be assumed otherwise based on the earlier code.
* | | | test/memory_usage_spec: skip on MacOS #15043Daniel Steinberg2021-07-10
| | | | | | | | | | | | Memory compression could complicate the measurements.
* | | | fix(lsp): Add separator between codelens in virttext (#15049)Mathias Fußenegger2021-07-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before: Lens1Lens2 After: Lens1 | Lens2 Fixes https://github.com/neovim/neovim/issues/15048
* | | | fix(lsp): correctly check for windows in lsp logger (#14954)Oliver Marriott2021-07-10
| | | |
* | | | Merge pull request #15047 from teto/flake-testsMatthieu Coudron2021-07-10
|\ \ \ \ | | | | | | | | | | flake update
| * | | | chore: add log hander to gen_vimdocMatthieu Coudron2021-07-10
| | | | | | | | | | | | | | | | | | | | | | | | | so it actually dumps output. Plus a description of what it does in the argparser.
| * | | | chore(flake): update inputsMatthieu Coudron2021-07-10
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated inputs so no need to add tree-sitter ourselves anymore. Added checks: - for pylint/shlint - distinguish the devolpment shell from the ASAN build (closure for ASAN version is smaller). While in the devShell, functionaltests would fail because bin/nvim could not load outputs/out/share/nvim/syntax/syntax.vim So we touch the file.
* | | | feat(lsp): Make line diagnostics display prettierMathias Fussenegger2021-07-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds indentation that matches the number prefix to ensure diagnostic messages spawning multiple lines align. Before: Diagnostics: 1. • Variable not in scope: red :: t0 -> t • Perhaps you meant one of these: ‘rem’ (imported from Prelude), ‘read’ (imported from Prelude), ‘pred’ (imported from Prelude) 2. • Variable not in scope: repeDoubleColon :: [Char] -> t0 • Perhaps you meant ‘replaceDoubleColon’ (line 32) After: Diagnostics: 1. • Variable not in scope: red :: t0 -> t • Perhaps you meant one of these: ‘rem’ (imported from Prelude), ‘read’ (imported from Prelude), ‘pred’ (imported from Prelude) 2. • Variable not in scope: repeDoubleColon :: [Char] -> t0 • Perhaps you meant ‘replaceDoubleColon’ (line 32)
* | | | Merge pull request #15036 from vigoux/decurbuf-2Björn Linse2021-07-09
|\ \ \ \ | | | | | | | | | | refactor(undo): don't assume curbuf in u_compute_hash
| * | | | refactor(undo): don't assume curbuf in u_compute_hashThomas Vigouroux2021-07-09
| | | | |
* | | | | Merge pull request #15035 from neovim/marvim/api-doc-update/masterThomas Vigouroux2021-07-09
|\ \ \ \ \ | |/ / / / |/| | | | docs: regenerate
| * | | | docs: regeneratemarvim2021-07-09
|/ / / /
* | | | fix(lsp): support duplicate params in signature help (#15032)Folke Lemaitre2021-07-09
| | | |
* | | | Merge pull request #12971 from vigoux/decurbufBjörn Linse2021-07-09
|\ \ \ \ | | | | | | | | | | Decrease reliance on curbuf in BUFEMPTY and `undo.c`
| * | | | refactor(undo): remove last occurence of save_curbufThomas Vigouroux2021-07-08
| | | | |
| * | | | refactor: remove all mentions to `curbuf_lock`Thomas Vigouroux2021-07-08
| | | | |
| * | | | style(undo): remove unnecessary commentThomas Vigouroux2021-07-06
| | | | |
| * | | | fix(qf): use correct buffer fieldThomas Vigouroux2021-07-06
| | | | |
| * | | | buffer: move BUFEMPTY to a functionThomas Vigouroux2021-07-06
| | | | |
| * | | | undo: reduce reliance on curbufThomas Vigouroux2021-07-06
| | | | |
| * | | | buffer: don't rely on curbuf in BUFEMPTYThomas Vigouroux2021-07-06
| | | | |
* | | | | Merge pull request #14908 from neovim/marvim/api-doc-update/masterJames McCoy2021-07-08
|\ \ \ \ \ | | | | | | | | | | | | docs: regenerate
| * | | | | docs: regeneratemarvim2021-07-08
| | | | | |
* | | | | | build(cmake): skip download if target file exists #14844)comicfans2021-07-08
| | | | | |
* | | | | | vim-patch:8.2.3125: variables are set but not used #15028dundargoc2021-07-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Variables are set but not used. Solution: Move the declarations to the block where they are used. (closes vim/vim#8527) https://github.com/vim/vim/commit/09f688c33aad9692276dfb68842cf0621a0e2002
* | | | | | vim-patch:8.2.3089: garbage collection has useless code #15027dundargoc2021-07-08
|/ / / / / | | | | | | | | | | | | | | | | | | | | Problem: Garbage collection has useless code. Solution: Bail out when aborting. (closes vim/vim#8504) https://github.com/vim/vim/commit/20cc52832067d22a3cd47dfb0805251228c32e7d
* | | | | fix(lsp): restore diagnostics extmarks that were moved to the last edit line ↵Folke Lemaitre2021-07-08
| | | | | | | | | | | | | | | | | | | | (#15023)
* | | | | feat(lsp): highlight active parameter in signature help (#15018)Folke Lemaitre2021-07-07
| | | | |