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

fix(installer): latest and specified release version for neovim have different urls #2484

Merged
merged 1 commit into from Apr 30, 2022

Conversation

lmckiwo
Copy link
Contributor

@lmckiwo lmckiwo commented Apr 19, 2022

Description

the URL used to download neovim are different if a version is specified vs if we leave it to default to the latest build.

Fixes #(issue)

How Has This Been Tested?

This script has been tested as follows:

RELEASE_VER=v0.6.1 bash /scripts/install-neovim-from-release
bash /scripts/install-neovim-from-release

and the nvim version verified after the installation.

@abzcoding abzcoding requested a review from kylo252 April 19, 2022 08:30
@kylo252 kylo252 changed the title [Bugfix]: latest build and specified release version for neovim has a different url fix(installer): latest and specified release version for neovim have different urls Apr 20, 2022
@kylo252
Copy link
Collaborator

kylo252 commented Apr 20, 2022

Thank you for your contribution! some minor fixes are needed to pass the CI tests

  1. commit-lint: "subject may not be empty [subject-empty]"
git commit --amend --no-edit --message "fix(installer): set correct url for nightly"
git push --force -u
  1. run shfmt: see https://github.com/LunarVim/LunarVim/blob/rolling/CONTRIBUTING.md#code-conventions

Check the contributing guidelines in https://github.com/LunarVim/LunarVim/blob/rolling/CONTRIBUTING.md.

Copy link
Collaborator

@kylo252 kylo252 left a comment

Choose a reason for hiding this comment

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

Thanks!

@kylo252 kylo252 merged commit ebc2987 into LunarVim:rolling Apr 30, 2022
tomazursic pushed a commit to tomazursic/LunarVim that referenced this pull request May 1, 2022
* upstream/rolling:
  feat: add cmp-tmux to the list of sources (LunarVim#2542)
  feat: gitsigns yadm support (LunarVim#2535)
  fix(installer): latest and specified release version for neovim have different urls (LunarVim#2484)
  fix(lsp): undo stdpath overload to avoid datarace (LunarVim#2540)
  feat(lua-dev): use the newer lua-dev branch till folke comes back (LunarVim#2538)
  fix(nvimtree): add latest changes from nvimtree (LunarVim#2537)
  feat(icons): make it possible to disable icons (LunarVim#2529)
  refactor(lsp): decouple the installer setup-hook (LunarVim#2536)
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

2 participants