freddiehaddad / feline.nvim

A minimal, stylish and customizable statusline, statuscolumn, and winbar for Neovim
GNU General Public License v3.0
289 stars 8 forks source link

bug: LuaRocks GHA failing on dependency. #73

Closed freddiehaddad closed 5 months ago

freddiehaddad commented 5 months ago

Did you check docs and existing issues?

Neovim version (nvim -v)

na

Operating system/version

na

Describe the bug

@mrcjkb github action is failing with:

Error: Failed installing dependency: https://luarocks.org/gitsigns.nvim-scm-1.rockspec - Build error: Failed installing lua/gitsigns/diff_ffi.lua in /home/runner/work/_temp/tmp.n5kdcAxHNW/lib/luarocks/rocks-5.1/gitsigns.nvim/scm-1/lua/gitsigns/diff_ffi.lua: lua/gitsigns/diff_ffi.lua: No such file or directory

https://github.com/freddiehaddad/feline.nvim/actions/runs/8754394451/job/24026039795#step:4:485

Looking into the problem, but maybe you can shed some light. I do see gitsigns registered with LuaRocks, so I don't think the issue is there.

Steps To Reproduce

Run the GHA workflow.

Expected Behavior

Successful push to LuaRocks repository.

Repro

na
freddiehaddad commented 5 months ago

@mrcjkb The wrong person got tagged in the above bug report. Tagging again here in case the update didn't send you a notification.

I did some reading on LuaRocks page. Do I need to create an account and upload a rockspec file? Not sure that's related to this issue though.

mrcjkb commented 5 months ago

Yep, gitsigns.nvim doesn't have a non-scm release yet (they're working on it). I would recommend waiting till it's been resolved (see https://github.com/lewis6991/gitsigns.nvim/pull/964#issuecomment-2061185572).

freddiehaddad commented 5 months ago

Sounds good. Created a PR #74 that adds the rockspec to the repo and updates the README. Will hold on merge until GitSigns issue is resolved.

mrcjkb commented 5 months ago

The rockspec isn't strictly necessary (luarocks-tag-release generates one based on GitHub repo metadata and the inputs), but it's still useful if you want to publish an scm/dev version.