search

AckslD / nvim-neoclip.lua

Clipboard manager neovim plugin with telescope integration
995 stars 20 forks source link

feat(length_limit): add length limit setting #60

Closed ouuan closed 2 years ago

ouuan commented 2 years ago

This resolves #55 and #59.

AckslD commented 2 years ago

Looks good @ouuan, but seems like some other tests are failing, do you know why?

ouuan commented 2 years ago

It even failed without my commits: https://github.com/ouuan/nvim-neoclip.lua/runs/6139514066

And it passed on my local machine.

ouuan commented 2 years ago

It's caused by updates of Telescope.

ouuan commented 2 years ago

Since the neovim version, plenary.nvim version, telescope.nvim version etc. changes every day, maybe we can have a daily scheduled CI run to catch dependencies' breaking changes.

AckslD commented 2 years ago

I see, I'll have to find some time to find out how to fix this, thanks for finding what broke!

AckslD commented 2 years ago

Fixed in #62

AckslD commented 2 years ago

Thanks a lot for the contribution @ouuan :tada: