nvim-neotest / neotest

An extensible framework for interacting with tests within NeoVim.
MIT License
2.12k stars 105 forks source link

fix: replace tbl_flatten to flatten():totable() #410

Closed pablobfonseca closed 1 month ago

pablobfonseca commented 1 month ago

This also fixes #408

rcarriga commented 1 month ago

Thanks for the PR!

codymikol commented 1 month ago

This introduced a regression for me,

neotest-kotest: ...local/share/nvim/lazy/neotest/lua/neotest/utils/init.lua:4: E5560: vimL function must not be called in a lua loop callback
stack traceback:
    ...ocal/share/nvim/lazy/neotest/lua/neotest/client/init.lua:89: in function <...ocal/share/nvim/lazy/neotest/lua/neotest/client/init.lua:88>
    [C]: in function 'has'
    ...local/share/nvim/lazy/neotest/lua/neotest/utils/init.lua:4: in function 'tbl_flatten'
    ...al/share/nvim/lazy/neotest/lua/neotest/lib/file/init.lua:358: in function 'match_root_pattern'
    /home/cody/dev/src/neotest-kotest/lua/neotest-kotest.lua:13: in function 'root'
    /home/cody/dev/src/neotest-kotest/lua/neotest-kotest.lua:115: in function 'build_spec'
    ...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:76: in function '_run_tree'
    ...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:65: in function <...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:22>
    [C]: in function 'xpcall'

I'll try and do some more digging

rcarriga commented 3 weeks ago

Should be fixed in latest commit :smile:

codymikol commented 3 weeks ago

thanks!