Munif Tanjim

Why would your express middleware arguments ever have anything after the `next`? The only two middleware signatures that express supports are: - `(req, res, next)` - `(err, req, res, next)`...

As far as I know, `express-async-handler` is only for using with **express middlewares**. > There are unit tests for it in this project. I noticed that too :thinking: It was...

Looks like `exrc` tests are skipped for now? 🤔 https://github.com/neovim/neovim/blob/23d8f5b870ab2a12882ba20e32d24b31c137f6a9/src/nvim/testdir/test_startup.vim#L1034 If testing is possible, I'd be happy to open another PR for adding tests. @zeertzjq

@jackMort can you try setting `grow = 1` instead of `size = "90%"` here? https://github.com/jackMort/ChatGPT.nvim/blob/d7703bd2453a1599367b3fd7df109601ee3783e4/lua/chatgpt/module.lua#L43

Can you check if the issue still exists with https://github.com/MunifTanjim/nui.nvim/pull/231?

Relevant Codes: - https://github.com/neovim/neovim/blob/bf0ac4f241cd22f53e026767ed5df89bf63bcf37/src/nvim/normal.c#L961-L962

- https://github.com/jackMort/ChatGPT.nvim/issues/20

Hey @linrongbin16 , I looked into your fork a few days ago. But couldn't figure out how to provide a function to dynamically set `remote`.

Okay, I reproduced it like this: ```lua vim.api.nvim_create_autocmd("BufWritePre", { callback = function() vim.wait(2000) end, }) ``` - Search for something using `*` or `/` - Clear highlight using `:nohl` -...

> I think all mentioned cases with catastrophic recursion actually have an issue in a grammars' composition and instead of attempts to handle unlimited recursions by queries it's better to...