aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | fix(ci): lower timeout to 45 minutes (from default 360 mins)Björn Linse2021-07-24
| | | | | | |
| * | | | | | fix(ci): disable broken test on openbsd on all CI due to resource constraintsBjörn Linse2021-07-23
|/ / / / / /
* | | | | | Merge pull request #15114 from theHamsta/treesitter-hl-priorityBjörn Linse2021-07-23
|\ \ \ \ \ \ | | | | | | | | | | | | | | feat(treesitter): allow to set highlight priority for queries
| * | | | | | docs(tree-sitter): document highlight priorityStephan Seitz2021-07-22
| | | | | | |
| * | | | | | feat(treesitter): allow to set highlight priority for queriesStephan Seitz2021-07-22
|/ / / / / /
* | | | | | lsp: add vim.lsp.diagnostic.set_qflist() function (#14831)Marc Jakobi2021-07-22
| | | | | | | | | | | | | | | | | | | | | | | | * Add vim.lsp.diagnostic.set_qflist() function * replaces opts.open_loclist with unified opts.open
* | | | | | Merge pull request #14771 from gpanders/colorschemeBjörn Linse2021-07-22
|\ \ \ \ \ \ | |/ / / / / |/| | | | | fix: source syncolors.vim before startup scripts
| * | | | | fix: source syncolors.vim before startup scriptsGregory Anders2021-07-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes an issue (#12573) where colorscheme files are sourced twice upon startup. This occurs when the startup script calls `:colorscheme`, which sets the `g:colors_name` global variable. When syntax highlighting is enabled in `syn_maybe_enable()` the `syntax.vim` script is sourced which in turn sources `synload.vim`. This script checks to see if `g:colors_name` is set and, if so, runs exe "colors " . colors_name This is done to ensure that highlight groups are defined before enabling the syntax highlighting engine. Instead, source syncolors.vim before the startup scripts which sets up default highlights and only load the full syntax engine after the startup scripts or when the user runs `:syntax on`. Add a guard variable `did_syncolor` to prevent syncolor.vim from being sourced twice and remove the line mentioned above from synload.vim so that the colorscheme file is not re-sourced when the syntax engine is loaded.
* | | | | | Revert "fix(ci): remove mongodb from brew to avoid brew upgrade error ↵Christian Clason2021-07-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (#15115)" (#15141) This reverts commit 5377b2b00aea1a0bde1b81452e6198dabe5b9796. (Fix no longer needed.)
* | | | | | lsp(start_client): Allow passing custom workspaceFolders to the LSP (#15132)sim2021-07-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some language servers *cough*rust-analyzer*cough* need an empty/custom workspaceFolders for certain usecases. For example, rust-analyzer needs an empty workspaceFolders table for standalone file support (See https://github.com/rust-analyzer/rust-analyzer/pull/8955). This can also be useful for other languages that need to commonly open a certain directory (like flutter or lua), which would help prevent spinning up a new language server altogether. In case no workspaceFolders are passed, we fallback to what we had before.
* | | | | | Merge pull request #15135 from neovim/marvim/api-doc-update/masterThomas Vigouroux2021-07-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | docs: regenerate
| * | | | | | docs: regeneratemarvim2021-07-19
|/ / / / / /
* | | | | | docs: made can_merge private (#15138)Folke Lemaitre2021-07-19
| | | | | |
* | | | | | fix(lsp): allow diagnostic.clear to accept nil bufnr (#15137)Gregory Anders2021-07-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Passing `nil` is equivalent to passing 0, i.e. it simply uses the current buffer number. This fixes a bug when vim.lsp.diagnostic.disable() is called without arguments.
* | | | | | Merge pull request #15127 from bfredl/nlua_initBjörn Linse2021-07-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | refactor(lua): initialize lua state at startup instead of dynamically
| * | | | | | refactor(lua): initialize lua state at startup instead of dynamicallyBjörn Linse2021-07-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | lua is used as part of implementation for more core features. As an example, every user keypress will invoke a lua function to check for keypress handlers (regardless if they are registered or not). Thus not starting lua until it is first used doesn't make much sense anymore. nlua_enter was also needed due to the earlier stateful &rtp translation, which by now have been made stateless.
* | | | | | | feat(lsp): allow diagnostics to be disabled for a buffer (#15134)Gregory Anders2021-07-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add two new methods to allow diagnostics to be disabled (and re-enabled) in the current buffer. When diagnostics are disabled they are simply not displayed to the user, but they are still sent by the server and processed by the client. Disabling diagnostics can be helpful in a number of scenarios. For example, if one is working on a buffer with an overwhelming amount of diagnostic warnings it can be helpful to simply disable diagnostics without disabling the LSP client entirely. This also allows users more flexibility on when and how they may want diagnostic information to be displayed. For example, some users may not want to display diagnostic information until after the buffer is first written.
* | | | | | | feat(lsp): allow configuring zindex for floating windows (#15086)rayx2021-07-19
| | | | | | |
* | | | | | | fix(shared): do not treat empty tables as list in deep extend (#15094)Folke Lemaitre2021-07-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An empty table was previously always treated as a list, which means that while merging tables, whenever an empty table was encountered it would always truncate any table on the left. `vim.tbl_deep_extend("force", { b = { a = 1 } }, { b = {} })` Before: `{ b = {} }` After: `{ b = { a = 1 } }`
* | | | | | | Merge pull request #15070 from dundargoc/PVS/V768James McCoy2021-07-19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | [RDY] chore(PVS/V768): don't use enum as bool.
| * | | | | | | chore(PVS/V768): don't use enum as bool.Dundar Göc2021-07-19
| |/ / / / / /
* | | | | | | Merge pull request #15073 from dundargoc/PVS/V751James McCoy2021-07-18
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | [RDY] chore(PVS/V751): tui.c, Parameter is not used
| * | | | | | | chore(PVS/V751): tui.c, Parameter is not usedJustin M. Keyes2021-07-18
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | False positive. Documentation for grid_scroll says "`cols` is always zero, reserved for future use".
* | | | | | | Merge pull request #15110 from dundargoc/PVS/V1042James McCoy2021-07-18
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | [RDY] chore: PVS/V1042 - ignore warning globally.
| * | | | | | chore: PVS/V1042 - ignore warning globally.Dundar Göc2021-07-17
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | V1042 is a warning that a file has a copyleft license, which is an irrelevant warning to open-source projects.
* / | | | | 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
generated by cgit (git 2.34.1) at 2025-06-25 05:13:29 +0000