Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add rockspec #2307

Merged
merged 1 commit into from
Dec 17, 2022
Merged

feat: add rockspec #2307

merged 1 commit into from
Dec 17, 2022

Conversation

mrcjkb
Copy link
Contributor

@mrcjkb mrcjkb commented Dec 11, 2022

nvim-lspconfig is a common dependency across Neovim plugins. Using luarocks may alleviate the need for users to specify their plugins' dependencies in their plugin manager.
(e.g., vim-plug or packer).

See also:
https://teto.github.io/posts/2021-09-17-neovim-plugin-luarocks.html

Copy link
Member

@justinmk justinmk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy to see this! Do we need to document how and when to bump the version? (May need to add a "Release" section to bottom of readme.)

@mrcjkb
Copy link
Contributor Author

mrcjkb commented Dec 11, 2022

Good point.
I have set the version to scm, which (afaict) tells luarocks to fetch the latest version from GitHub.

If we want to configure the rockspec to align with the release versions, I would suggest working towards a github actions workflow that automatically creates a PR with a rockspec version bump on release.

nvim_lspconfig-scm-1.rockspec Outdated Show resolved Hide resolved
@teto
Copy link
Member

teto commented Dec 12, 2022

@justinmk the only issue is to deal with who is going to update the package on www.luarocks.org since it can be done from only one account. See luarocks/luarocks-site#173.

@mrcjkb thanks for doing this !

nvim-lspconfig is a common dependency across Neovim plugins.
Using luarocks may alleviate the need for users to specify their
plugins' dependencies in their plugin manager.
(e.g., vim-plug or packer).

See also:
https://teto.github.io/posts/2021-09-17-neovim-plugin-luarocks.html
@teto teto merged commit 54eb2a0 into neovim:master Dec 17, 2022
@teto
Copy link
Member

teto commented Dec 17, 2022

added to https://luarocks.org/labels/neovim

@mrcjkb mrcjkb deleted the rockspec branch December 17, 2022 16:09
mrcjkb added a commit to mrcjkb/nvim-lspconfig that referenced this pull request Jan 6, 2023
The recently added rockspec (see neovim#2307) has to be prepared and
uploaded to luarocks for each release.

This adds an automated workflow for it, which has been tested with
plenary.nvim.

To be able to upload to luarocks, the owner of the luarocks account
will have to add an API key named `LUAROCKS_API_KEY` to this repo's
GitHub Actions secrets.
teto pushed a commit that referenced this pull request Jan 8, 2023
The recently added rockspec (see #2307) has to be prepared and
uploaded to luarocks for each release.

This adds an automated workflow for it, which has been tested with
plenary.nvim.

To be able to upload to luarocks, the owner of the luarocks account
will have to add an API key named `LUAROCKS_API_KEY` to this repo's
GitHub Actions secrets.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants