aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | Merge pull request #22194 from bfredl/noflushbfredl2023-02-10
|\ \ \ \ \ | |/ / / / |/| | | | refactor(ui): remove some superfluous redraw and ui_flush() calls
| * | | | refactor(ui): remove some superfluous ui_flush() callsbfredl2023-02-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - <expr> mapping has no business saving and restoring the low-level UI cursor. The cursor will be put in a reasonable position after input is processed, chill out. - TUI handles output needed for suspend - vgetc() family of function does flushing
| * | | | refactor(ui): don't reimplement redrawing in focus gained handlingbfredl2023-02-09
| | | | | | | | | | | | | | | | | | | | | | | | | These are just ordinary boring events now. Modes already redraw events themselves.
* | | | | ci: simplify lintcommit output (#22204)dundargoc2023-02-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Suggest reading CONTRIBUTING.md once, not for each commit failure - Suggest using "fix" type if none of the provided types are appropriate - Remove "dist" type. It's rarely used and can be replaced by using the "build" type
* | | | | ci: remove unnecessary environment variables (#22175)dundargoc2023-02-10
| | | | |
* | | | | ci: split functionaltest-lua into two separate jobs (#22201)dundargoc2023-02-10
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | More specifically, move the job testing the oldest supported cmake into its own job. This opens the way for other jobs to use powerful and advanced cmake features such as choosing which files to use with the -S flag. Removed testing from this job as this probably won't reveal anything that other jobs already doesn't already show, since the only difference is the cmake version.
* | | | ci: inline test.sh to run_tests.sh (#22198)dundargoc2023-02-10
| | | | | | | | | | | | This will get rid of the common/ directory.
* | | | ci(lintcommit): turn off debug tracing (#22196)dundargoc2023-02-09
| | | | | | | | | | | | | | | | The default output is too verbose and messy for someone not already familiar with lintcommit, which defeats it purpose.
* | | | ci: remove base branch from cache key (#22195)dundargoc2023-02-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the base branch as cache means that pull requests won't be able to use the cache from the master branch, since the master branch cache doesn't have a base_ref as it's generated from a push. Removing base_ref makes the cache key from master and PR branch the same, provided the any build files don't change.
* | | | ci: inline internal caching script to Github actions (#22192)dundargoc2023-02-09
|/ / / | | | | | | | | | | | | I don't think it's possible to meaningfully abstract away caching on multiple providers, as each provider has different mechanisms on how they work.
* | | ci: add individual timeout limits for all tests (#22193)dundargoc2023-02-09
| | | | | | | | | | | | | | | The CI somtimes freezes on a specific test, wasting 45 minutes for the entire job. Adding a timeout of 15 minutes to functionaltest and 5 minutes to unittests will mitigate the problem.
* | | Merge pull request #22186 from clason/maintainChristian Clason2023-02-09
|\ \ \ | | | | | | | | | | | | * docs(maintain): add note on updating `luvref.txt` * docs(luvref): update version info
| * | | docs(luvref): update version infoChristian Clason2023-02-09
| | | |
| * | | docs(maintain): add note on updating luvref.txtChristian Clason2023-02-09
| | | |
* | | | test(exit_spec): make sure that autocommands are triggered (#22188)zeertzjq2023-02-09
| | | | | | | | | | | | | | | | | | | | Previously, if the autocommands are not triggered, the tests may still pass because no assertion is done. Add an assertion so that the tests will fail if the autocommands aren't triggered.
* | | | build: remove codecov related files (#20859)dundargoc2023-02-09
| | | | | | | | | | | | These aren't needed as we don't use codecov anymore.
* | | | fix(rpc): ignore redraw events when exiting (#22184)zeertzjq2023-02-09
| | | | | | | | | | | | | | | | When a TUI client has already stopped, handling UI events will cause a heap-use-after-free, so ignore them.
* | | | fix(rpc): ignore redraw events when not in UI client (#21892)zeertzjq2023-02-09
| | | | | | | | | | | | Otherwise it will crash.
* | | | build: create test/CMakeLists.txt and move test-related code (#22179)dundargoc2023-02-08
| | | | | | | | | | | | | | | | | | | | Having a clear separation between build code and test code makes it easier to get a higher-level understanding of how the neovim build works.
* | | | build: reuse source files with interface library (#22177)dundargoc2023-02-08
| | | | | | | | | | | | This will minimize duplication by only needing to specify required files for nvim and libnvim once.
* | | | Merge pull request #22172 from bfredl/cellsbfredl2023-02-08
|\ \ \ \ | | | | | | | | | | perf(ui): mitigate redraw latency regression from TUI refactor
| * | | | perf(ui): mitigate redraw latency regression from TUI refactorbfredl2023-02-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While the new RPC encoder/decoder implementation in general should have less overhead than the deleted UIBridge serializer previously used for the TUI, it regresses on redraw latency in one important aspect. The old bridge implementation allowed the TUI to process a previous screen line internally in parallel with the main thread rendering the next one in win_line etc. As printing the escape sequences in highlighted cells has a considerable hit in profiles, this has a substantial effect on redraw latency. The RPC implementation, however, waits with sending any data until either a flush, or the buffer is full. This change lowers the granularity of communication again, using an adjustable threshold counted in number of cell events (discounting long repeats and clearing as maximum a single extra event). The current value is guesstimated to something simple on a reasonable scale, which should be bigger than a single line, but multiple events for a big multi-window screen.
| * | | | refactor(ui): cleanup 'redrawdebug', introduce "flush" modebfredl2023-02-08
| | | | |
* | | | | ci: remove fail summary (#22174)dundargoc2023-02-08
| | | | | | | | | | | | | | | | | | | | The tests already have a summary at the end, there's no need for an additional fail summary wrapper.
* | | | | build: remove unused function get_test_target (#22176)dundargoc2023-02-08
| | | | |
* | | | | build: replace check-single-includes with clang-tidy (#22061)dundargoc2023-02-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clang-tidy already does what check-single-includes does automatically on top of its regular linting. It is also generator independent, so it doesn't take an eternity to run on slower generators such as Visual Studio.
* | | | | build(windows): specify Windows 8 as the minimum version (#22173)dundargoc2023-02-08
| | | | | | | | | | | | | | | | | | | | This will allow MSVC to use newer features not available in Vista and Windows 7.
* | | | | ci: remove unnecessary END_MARKER variable (#22171)dundargoc2023-02-08
| | | | |
* | | | | ci(oldtest): make a copy of scripts of ci/common in testdir (#22170)dundargoc2023-02-08
| | | | | | | | | | | | | | | | | | | | Having separate copies makes it easier to not accidentally break something when modifying the scripts.
* | | | | Merge pull request #22165 from clason/bump-LuvChristian Clason2023-02-08
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | build(deps): bump Luv to HEAD - e8e7b7e13 docs(luvref): update to latest version
| * | | | | docs(luvref): update to version bumpChristian Clason2023-02-08
| | | | | |
| * | | | | build(deps): bump Luv to HEAD - e8e7b7e13Christian Clason2023-02-08
| | | | | |
* | | | | | build(deps): bump libuv to HEAD - 62c2374a8 (#22166)Christian Clason2023-02-08
| | | | | |
* | | | | | build: prefer -D <variable>=<value> over -D<variable>=<value> (#22164)dundargoc2023-02-08
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | This makes it easier to see that -D is referring to the entire "<variable>=<value>", rather than only <variable>. It also help syntax highlighters highlight built-in variables.
* | | | | build(Windows): make bundling nvim-qt optional (#21866)Enan Ajmain2023-02-08
| |/ / / |/| | | | | | | Closes https://github.com/neovim/neovim/issues/14552.
* | | | vim-patch:9.0.1291: Move language files are not recognized (#22162)Christian Clason2023-02-08
|/ / / | | | | | | | | | | | | | | | | | | | | | Problem: Move language files are not recognized. Solution: Recognize Move language files. (Amaan Qureshi, closes vim/vim#11947) https://github.com/vim/vim/commit/6642982beaf4f1f5164f0315a1b3e3c275156089 Co-authored-by: Amaan Qureshi <amaanq12@gmail.com>
* | | ci: add universal macos job (#22156)dundargoc2023-02-07
| | | | | | | | | | | | | | | The universal macos release is particularly sensitive to build system changes. Adding a job that builds a universal binary whenever a cmake file is changed will help prevent future release breaks.
* | | build: remove duplicate INTERFACE keyword (#22106)dundargoc2023-02-07
| | |
* | | ci: simplify how environment variables are used (#22067)dundargoc2023-02-07
| | | | | | | | | | | | Having a clear separation between when we manipulate variables and when we export them to GITHUB_ENV makes it less error-prone.
* | | vim-patch:9.0.1288: FunC files are not recognized (#22153)Christian Clason2023-02-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: FunC files are not recognized. Solution: Recognize FunC files. (Amaan Qureshi, closes vim/vim#11949) https://github.com/vim/vim/commit/91deac45392fe93094b9c31403b1ae771dc71938 Co-authored-by: Amaan Qureshi <amaanq12@gmail.com>
* | | ci: remove unnecessary variables and functions (#22150)dundargoc2023-02-07
| | |
* | | vim-patch:9.0.1290: CTRL-N and -P on cmdline don't trigger CmdlineChanged ↵zeertzjq2023-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (#22151) Problem: CTRL-N and -P on cmdline don't trigger CmdlineChanged. Solution: Jump to cmdline_changed instead of cmdline_not_changed. (closes vim/vim#11956) https://github.com/vim/vim/commit/af9e28a5b8f888b79459393ddb26fffe613c3f3c Cherry-pick Test_Cmdline() change from patch 9.0.1039.
* | | build: don't build libnvim when running the CI (#22149)dundargoc2023-02-06
| | | | | | | | | | | | It shouldn't be sneakily run alongside the nvim build. If it's to be used it should be done in a separate step.
* | | ci: remove unhelpful helper functions for make (#22148)dundargoc2023-02-06
| | |
* | | feat(spell): also source `spell/LANG.lua` when setting `spelllang` (#22111)Christian Clason2023-02-06
| | | | | | | | | | | | | | | | | | Problem: only Vimscript files are sourced when setting spelling language Solution: also source Lua files after sourcing Vimscript files Closes #22110
* | | vim-patch:9.0.1282: Ron files are not recognized (#22132)Amaan Qureshi2023-02-06
| | | | | | | | | | | | | | | | | | Problem: Ron files are not recognized. Solution: Recognize Ron files. (Amaan Qureshi, closes vim/vim#11948) https://github.com/vim/vim/commit/c8ef30bc2eaec956549510cd4b2efc96b7aee563
* | | fix(health): iterate using ipairs correctly (#22119)Mateusz Majewski2023-02-06
| | | | | | | | | | | | | | | | | | In a few places ipairs was used to iterate over elements of the array. However, the first return value of ipairs was erronously used, which is not the value, but rather the index. This would result in errors, for instance when trying to retrieve a field from the value.
* | | fix(decoration): don't show signcolumn for non-sign_text extmark (#22135)Lewis Russell2023-02-05
| | | | | | | | | Fixes: #22127
* | | Merge pull request #21548 from figsoda/transform-captureLewis Russell2023-02-05
|\ \ \ | | | | | | | | feat(treesitter): allow capture text to be transformed
| * | | feat(treesitter): respect metadata[id].range for offset!figsoda2023-02-04
| | | |