aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | | | | | | | 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.
* | | | | | | | | | | | Merge pull request #17000 from dundargoc/ci/use-continue-on-errorJames McCoy2022-01-08
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | | ci: use continue-on-error instead of "|| true"
| * | | | | | | | | | | ci: use continue-on-error instead of "|| true"Dundar Göc2022-01-08
| |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The intention is clearer and doesn't rely on shell-isms.
* | | | | | | | | | | Merge pull request #16979 from dundargoc/ci/add-tsan-clang-sanitizerJames McCoy2022-01-08
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | ci: set CLANG_SANITIZER variable to TSAN when running the tsan job
| * | | | | | | | | | | ci: set CLANG_SANITIZER variable to TSAN when running the tsan jobDundar Göc2022-01-07
| | |_|/ / / / / / / / | |/| | | | | | | | |
* | | | | | | | | | | fix(lsp): only send valid params in executeCommand (#16987)Michael Lingelbach2022-01-08
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #16848 from dundargoc/refactor/prevent-overflow-by-castingbfredl2022-01-08
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | refactor: avoid overflow by explicitly casting operand to a wider type
| * | | | | | | | | | | chore(fixup): bfredl size_tDundar Göc2022-01-01
| | | | | | | | | | | |
| * | | | | | | | | | | chore(fixup): zeerdundargoc2022-01-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: zeertzjq <zeertzjq@outlook.com>
| * | | | | | | | | | | refactor: avoid overflow by explicitly casting operand to a wider typeDundar Göc2021-12-31
| | | | | | | | | | | |
* | | | | | | | | | | | test: remove redundant feed_command('terminal') (#16993)zeertzjq2022-01-08
| |_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | It just types :terminal<CR> into the terminal, which is redundant.
* | | | | | | | | | | Merge pull request #16991 from bk2204/ex-modebfredl2022-01-08
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Make mode() return correct value in ex mode
| * | | | | | | | | | | fix: make mode() return correct value in ex modebrian m. carlson2022-01-08
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the user is in ex mode, a call to mode(1) is documented to return "cv". However, it does not currently do so, because the check which checks for ex mode is nested inside a conditional which is never reached in ex mode. Vim uses an explicit check for exmode_active, so let's do the same thing here. Add some tests for this case both with a TTY and in silent mode.
* | | | | | | | | | | fix(lsp): resolve bufnr for get_lines (#16986)Michael Lingelbach2022-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes https://github.com/neovim/neovim/issues/16985 * get_lines checks if buf_loaded using bufnr 0, which is typically used as a sentinel value, but here must be resolved to the true bufnr
* | | | | | | | | | | fix(filetype): match negative priority patterns after extensions (#16980)Gregory Anders2022-01-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Negative priority patterns are those that act as catch-alls when all other attempts at matching have failed (typically the patterns that use the StarSetf functions).
* | | | | | | | | | | vim-patch:8.2.4033: running filetype tests leaves directory behind (#16984)zeertzjq2022-01-07
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Running filetype tests leaves directory behind. Solution: Delete the top directory. (closes vim/vim#9483) https://github.com/vim/vim/commit/a4c96252b12c9ebc0ba563694c064e500d707b06
* | | | | | | | | | Merge pull request #16970 from dundargoc/ci/serialize-labeler-workflowJames McCoy2022-01-07
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | ci: run type-scope job after the triage jobDundar Göc2022-01-07
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should prevent the scenario of one job accidentally removing a relevant label from the other.
* | | | | | | | | | feat(lsp): skip or reduce debounce after idle (#16881)Mathias Fußenegger2022-01-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The idea of the debounce is to avoid overloading a server with didChange notifications. So far this used a constant value to group changes within an interval together and send a single notification. A side effect of this is that when you were idle, notifications are still delayed. This commit changes the logic to take the time the last notification happened into consideration, if it has been greater than the debounce interval, the debouncing is skipped or at least reduced.
* | | | | | | | | | vim-patch:8.2.4014: git and gitcommit file types not properly recognized ↵Christian Clason2022-01-07
| |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (#16953) Problem: Git and gitcommit file types not properly recognized. Solution: Adjust filetype detection. (Tim Pope, closes vim/vim#9477) https://github.com/vim/vim/commit/c689f8c3d98fffe7e13730e198ce120934528f9c
* | | | | | | | | vim-patch:8.2.0951: search stat test has leftover from debugging (#16967)zeertzjq2022-01-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Search stat test has leftover from debugging. Solution: Remove line that writes a file. (Christian Brabandt, closes vim/vim#6224) https://github.com/vim/vim/commit/6ba24d87630b1ec2b8c7ff71550c9e41d143800e
* | | | | | | | | fix(runtime): check markdownError before syn-clear (#16930)Frederick Zhang2022-01-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoids `E28: No such highlight group name: markdownError` when using a foreign markdown syntax.
* | | | | | | | | fix(lua): print multiple return values with =expr (#16933)Shadman2022-01-06
| | | | | | | | |
* | | | | | | | | feat(lua): add notify_once() (#16956)Gregory Anders2022-01-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | Like vim.notify(), but only displays the notification once.
* | | | | | | | | Merge pull request #16591 from shadmansaleh/feat/lua_keymaps2Björn Linse2022-01-06
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | feat(lua): add support for lua keymaps
| * | | | | | | | | feat(lua): add vim.keymapshadmansaleh2022-01-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This introduces two new functions `vim.keymap.set` & `vim.keymap.del` differences compared to regular set_keymap: - remap is used as opposite of noremap. By default it's true for <Plug> keymaps and false for others. - rhs can be lua function. - mode can be a list of modes. - replace_keycodes option for lua function expr maps. (Default: true) - handles buffer specific keymaps Examples: ```lua vim.keymap.set('n', 'asdf', function() print("real lua function") end) vim.keymap.set({'n', 'v'}, '<leader>lr', vim.lsp.buf.references, {buffer=true}) vim.keymap.set('n', '<leader>w', "<cmd>w<cr>", {silent = true, buffer = 5 }) vim.keymap.set('i', '<Tab>', function() return vim.fn.pumvisible() == 1 and "<C-n>" or "<Tab>" end, {expr = true}) vim.keymap.set('n', '[%', '<Plug>(MatchitNormalMultiBackward)') vim.keymap.del('n', 'asdf') vim.keymap.del({'n', 'i', 'v'}, '<leader>w', {buffer = 5 }) ```
* | | | | | | | | | Merge pull request #16817 from lewis6991/masterBjörn Linse2022-01-06
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | | vim-patch:8.2.3921: the way xdiff is used is inefficient
| * | | | | | | | | vim-patch:8.2.3921: the way xdiff is used is inefficientLewis Russell2021-12-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The way xdiff is used is inefficient. Solution: Use hunk_func instead of the out_line callback. (Lewis Russell, closes vim/vim#9344) https://github.com/vim/vim/commit/d9da86e94ea8dbaa056270a666892945c40a6674
* | | | | | | | | | Merge pull request #16942 from gpanders/test-filetype-luaGregory Anders2022-01-05
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | fix(filetype): normalize slashes in file pathsGregory Anders2022-01-05
| | | | | | | | | | |
| * | | | | | | | | | test(filetype): port test_filetype to LuaGregory Anders2022-01-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because filetype.lua is gated behind an opt-in variable, it's not tested during the "standard" test_filetype.vim test. So port the test into filetype_spec where we enable the opt-in variable. This means runtime Vim patches will need to update test_filetype in two places. This can eventually be removed if/when filetype.lua is made opt-out rather than opt-in.
* | | | | | | | | | | chore: remove lspconfig checkhealth from issue template (#16948)Michael Lingelbach2022-01-05
|/ / / / / / / / / /
* | | | | | | | | | fix(filetype): match on <afile> rather than <abuf> (#16943)Gregory Anders2022-01-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Filetype detection runs on BufRead and BufNewFile autocommands, both of which can fire without an underlying buffer, so it's incorrect to use <abuf> to determine the file path. Instead, match on <afile> and assume that the buffer we're operating on is the current buffer. This is the same assumption that filetype.vim makes, so it should be safe.
* | | | | | | | | | feat(lsp): enable default debounce of 150 ms (#16908)Michael Lingelbach2022-01-05
| |_|_|_|_|/ / / / |/| | | | | | | |
* | | | | | | | | feat(autocmd): populate v:event in RecordingLeave (#16828)Axel Dahlberg2022-01-04
| | | | | | | | |
* | | | | | | | | feat(filetype.lua): add support for txt files (#16926)KillTheMule2022-01-04
| | | | | | | | |
* | | | | | | | | chore: ignore unused args in filetype functions (#16927)Gregory Anders2022-01-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | Also mark the 'getline' helper function as private to avoid docgen.
* | | | | | | | | test: fix absolute paths in filetype_spec (#16920)Gregory Anders2022-01-04
| | | | | | | | |
* | | | | | | | | fix(filetype): set default ft_ignore_pat in filetype.lua (#16917)Gregory Anders2022-01-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This default value is also set in filetype.vim, but if filetype.vim is disabled the variable is never defined, which causes errors in some of the dist#ft detection functions.
* | | | | | | | | feat(filetype.lua): Add typescript extension to filetype detection (#16923)Robin Gagnon2022-01-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | port from `filetype.vim`; also add `getline` convenience function
* | | | | | | | | fix(api): force redefinition of user commands by default (#16918)Gregory Anders2022-01-04
| | | | | | | | |