aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | | | | | | | Merge pull request #16978 from gpanders/filetype-updatesGregory Anders2022-01-13
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | fix(filetype): fix foam pattern detectionGregory Anders2022-01-13
| | | | | | | | | | | |
| * | | | | | | | | | | feat(filetype.lua): fix .cc file not detectedrhcher2022-01-13
| | | | | | | | | | | |
| * | | | | | | | | | | feat(filetype.lua): add support for files under .gitSanchayan Maity2022-01-13
| | | | | | | | | | | |
| * | | | | | | | | | | feat(filetype.lua): add support for patch filesSanchayan Maity2022-01-13
| | | | | | | | | | | |
| * | | | | | | | | | | feat(filetype.lua): add support for tmux.conf filesGary Sentosa2022-01-13
| | | | | | | | | | | |
| * | | | | | | | | | | feat(filetype.lua): fix .env file not detectedGary Sentosa2022-01-13
|/ / / / / / / / / / /
* | | | | | | | | | | vim-patch:8.2.4077: not all Libsensors files are recognized (#17080)Christian Clason2022-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Not all Libsensors files are recognized. Solution: Add "sensors.d/*" pattern. (Doug Kearns) https://github.com/vim/vim/commit/8d9e470aa91a93da7d6bda62521aef69a79e956d
* | | | | | | | | | | fix(lsp): forward offset_encoding in rename handler (#17079)Michael Lingelbach2022-01-13
| | | | | | | | | | |
* | | | | | | | | | | fix(lsp): forward offset_encoding to apply_text_edits (#17075)Michael Lingelbach2022-01-13
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #17034 from zeertzjq/compl-lua-selectbfredl2022-01-13
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | feat(completion): support selecting item via API from Lua mapping
| * | | | | | | | | | | feat(completion): support selecting item via API from Lua mappingzeertzjq2022-01-11
| | | | | | | | | | | |
* | | | | | | | | | | | fix(lsp): strictly enforce passing offset encoding (#17049)Michael Lingelbach2022-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the "fallback" to utf-16 in many of our helper functions. We should always explicitly pass these around when possible except in two locations: * generating params with help utilities called by buf.lua functions * the buf.lua functions themselves Anything that is called by the handler should be passed the offset encoding.
* | | | | | | | | | | | fix(lsp): handle negative activeSignature in signatureHelp (#17064)Mathias Fußenegger2022-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | omnisharp-roslyn can send negative values: { activeParameter = 0, activeSignature = -1, signatures = { { documentation = "", label = "TestEntity.TestEntity()", parameters = {} } } } In 3.16 of the specification `activeSignature` is defined as `uinteger` and therefore negative values shouldn't be allowed, but within 3.15 it was defined as `number` which makes me think we can be a bit lenient in this case and handle them. The expected behavior is quite clear: The active signature. If omitted or the value lies outside the range of `signatures` the value defaults to zero or is ignored if the `SignatureHelp` has no signatures. Fixes an error: util.lua:1685: attempt to get length of local 'lines' (a nil value) util.lua:1685: in function 'trim_empty_lines' handlers.lua:334: in function 'textDocument/signatureHelp'
* | | | | | | | | | | | fix(lsp): fix applying multiple out-of-range TextEdits (#17037)hrsh7th2022-01-13
| |_|_|_|/ / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #16945 from theHamsta/cached-queriesThomas Vigouroux2022-01-13
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | perf(treesitter): cache query parsing
| * | | | | | | | | | | perf(treesitter): cache query parsingStephan Seitz2022-01-06
| | | | | | | | | | | |
* | | | | | | | | | | | fix(keywordprg): retain terminal buffer after K (#17046)Daniel Steinberg2022-01-12
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #17058 from clason/bump-treesitterThomas Vigouroux2022-01-12
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | build(deps): bump treesitter to commit bf210f0
| * | | | | | | | | | | | build(deps): bump treesitter to commit bf210f0Christian Clason2022-01-12
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update to https://github.com/tree-sitter/tree-sitter/commit/bf210f0c9ec7931c1a5f639461495db240aac149, which is a significant improvement to query performance.
* | | | | | | | | | | | Merge pull request #17054 from jamessan/debug-publish-issuesJames McCoy2022-01-12
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | ci(release): enable gh's api debug logs when creating the release
| * | | | | | | | | | | | ci(release): enable gh's api debug logs when creating the releaseJames McCoy2022-01-12
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is intended to help track down why the release is sporadically left in draft state, rather than being published. [skip ci]
* | | | | | | | | | | | Merge pull request #17045 from gpanders/diagnostic-fixupsGregory Anders2022-01-11
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / |/| | | | | | | | | | | Various diagnostic fixups
| * | | | | | | | | | | feat(diagnostic): allow retrieving current diagnostic configGregory Anders2022-01-11
| | | | | | | | | | | |
| * | | | | | | | | | | fix(diagnostic): only set default handler config if unsetGregory Anders2022-01-11
| | | | | | | | | | | |
| * | | | | | | | | | | fix(diagnostic): allow setting arbitrary config valuesGregory Anders2022-01-11
| | | | | | | | | | | |
| * | | | | | | | | | | fix(diagnostic): resolve nil opts tablesGregory Anders2022-01-11
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In functions which allow opts to be optional, ensure that the value actually resolves to a non-nil value.
* | | | | | | | | | | vim-patch:8.2.4064: foam files are not detected (#17041)Christian Clason2022-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * vim-patch:8.2.4064: foam files are not detected Problem: Foam files are not detected. Solution: Detect the foam filetype by the path and file contents. (Mohammed Elwardi Fadeli, closes vim/vim#9501) https://github.com/vim/vim/commit/2284f6cca384e0ccc352bfec7277dc26386cac3d * Port foam ft detection to filetype.lua Co-authored-by: Gregory Anders <greg@gpanders.com>
* | | | | | | | | | | refactor(lsp): debounce timer per buf and unify with non-debounce (#17016)Mathias Fußenegger2022-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of the `pending_change` closure in the `changetracking.prepare` was a bit confusing because it has access to `bufnr` and `uri` but it could actually contain pending changes batched for multiple buffers. (We accounted for that by grouping `pending_changes` by a `uri`, but it's not obvious what's going on) This commit changes the approach to do everything per buffer to avoid any ambiguity. It also brings the debounce/no-debounce a bit closer together: The only difference is now whether a timer is used or if it is triggered immediately
* | | | | | | | | | | Merge pull request #16962 from f380cedric/vim-8.2.3409Thomas Vigouroux2022-01-11
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vim-patch:8.2.3409
| * | | | | | | | | | | vim-patch:8.2.3409: reading beyond end of line with invalid utf-8 characterf380cedric2022-01-06
| |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Reading beyond end of line with invalid utf-8 character. Solution: Check for NUL when advancing. https://github.com/vim/vim/commit/65b605665997fad54ef39a93199e305af2fe4d7f Dropped the non-test part, since it is only non-utf8 related.
* | | | | | | | | | | vim-patch:2f0936cb9a2e (#17007)Christian Clason2022-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * vim-patch:2f0936cb9a2e Update runtime files https://github.com/vim/vim/commit/2f0936cb9a2eb026acac03e6a8fd0b2a5d97508b
* | | | | | | | | | | Merge pull request #16961 from f380cedric/vim-8.2.3582Thomas Vigouroux2022-01-11
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | | vim-patch:8.2.3582
| * | | | | | | | | | vim-patch:8.2.3582: reading uninitialized memory when giving spell suggestionsf380cedric2022-01-06
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Reading uninitialized memory when giving spell suggestions. Solution: Check that preword is not empty. https://github.com/vim/vim/commit/15d9890eee53afc61eb0a03b878a19cb5672f732
* | | | | | | | | | fix(man.vim): fix search function on some systems (#13709)Will Eccles2022-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes man.vim's searching on some systems (namely mandoc) where previously it would not respect the value of b:man_default_sects. It now properly parses man pages on these systems.
* | | | | | | | | | fix(lsp): ensure pending changes are flushed on skipped debounce (#17015)Mathias Fußenegger2022-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow up to https://github.com/neovim/neovim/pull/16881 Document changes could get sent out of order to the server: 1. on_lines: debounce > 0; add to pending changes; setup timer 2. on_lines: debounce = 0; send new changes immediately 3. timer triggers, sending changes from 1.
* | | | | | | | | | Merge pull request #17025 from zeertzjq/compl-k-luabfredl2022-01-10
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | fix(completion): prevent K_LUA from closing pum
| * | | | | | | | | | fix(completion): prevent K_LUA from closing pumzeertzjq2022-01-10
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #17006 from vigoux/tree-sitter-allocThomas Vigouroux2022-01-10
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | | feat(treesitter): set allocator when possible
| * | | | | | | | | | feat(treesitter): set allocator when possibleThomas Vigouroux2022-01-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a new cmake check to keep this backwards compatible with the different versions of tree-sitter.
* | | | | | | | | | | Merge pull request #17013 from dundargoc/ci/continue-on-errorJames McCoy2022-01-09
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | ci: add a continue-on-error to the add-reviewer jobs
| * | | | | | | | | | | ci: add a continue-on-error to the add-reviewer jobsDundar Göc2022-01-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is to circumvent a limitation in GitHub Actions that requires special organization access in order to add any reviewers.
* | | | | | | | | | | | Merge pull request #16849 from dundargoc/refactor/pvs/v1048bfredl2022-01-09
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | refactor(PVS/V1048): prevent "variable was assigned the same value" warning
| * | | | | | | | | | | refactor(PVS/V1048): ignore "assigned the same value" warningDundar Göc2022-01-08
| | | | | | | | | | | |
* | | | | | | | | | | | test: use old style test for testing filetype.lua (#17003)Gregory Anders2022-01-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a much better solution than #16942 as it doesn't require copying every new change from test_filetype.vim into filetype_spec.lua (which is much more maintainable).
* | | | | | | | | | | | fix(api): validate user_command name (#17004)Lewis Russell2022-01-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The name argument of nvim_add_user_command must begin with an uppercase character. Check that is does.
* | | | | | | | | | | | Merge pull request #16965 from dundargoc/ci/add-reviewer-based-on-labelJames McCoy2022-01-09
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | | ci: add reviewer based on label
| * | | | | | | | | | | ci: add reviewer based on labelDundar Göc2022-01-09
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also remove CODEOWNERS since this is a superior solution.
* | | | | | | | | | | Merge pull request #16983 from dundargoc/test/add-vimruntime-variableJames McCoy2022-01-08
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | test: add VIMRUNTIME variable to the clear function
| * | | | | | | | | | | test: add VIMRUNTIME variable to the clear functionDundar Göc2022-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some of the tests will fail if this isn't explicitly set.