search

kndndrj / nvim-dbee

Interactive database client for neovim
GNU General Public License v3.0
617 stars 40 forks source link

Use `go build` instead of `go install` #29

Closed 33KK closed 11 months ago

33KK commented 11 months ago

Doesn't require the CI pipeline to work, and easier to verify that binary matches source code.

Not tested on windows, not sure if the path needs to be normalized. Seems to work, at least in Git Bash, and whatever winget install Neovim.Neovim installed.

kndndrj commented 11 months ago

I had a similar idea when I did the installation script, but I was a bit hesitant to rely on the debug api.

But now I checked into nvim's history and it seems pretty stable, so I'll test this a bit and then merge.

kndndrj commented 11 months ago

Verified that it works, windows support is not something which is super important atm, since I expect a lot of stuff needs to be fixed before we get propper windows support.

Also thanks a lot for the contributions!