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

Address deprecation warnings #298

Merged
merged 2 commits into from
Apr 5, 2023
Merged

Address deprecation warnings #298

merged 2 commits into from
Apr 5, 2023

Conversation

trekie86
Copy link

@trekie86 trekie86 commented Apr 4, 2023

This change addresses deprecation warnings and updates cmp_nvim_lsp per hrsh7th/cmp-nvim-lsp#38. Additional migration from sumneko_lua to lua_ls and the corresponding settings file.

fixes #294

This change addresses deprecation warnings and updates cmp_nvim_lsp per
hrsh7th/cmp-nvim-lsp#38. Additional migration
from sumneko_lua to lua_ls and the corresponding settings file
This change addresses deprecation warnings and updates cmp_nvim_lsp per
hrsh7th/cmp-nvim-lsp#38. Additional migration
from sumneko_lua to lua_ls and the corresponding settings file
Copy link
Collaborator

@gnmearacaun gnmearacaun left a comment

Choose a reason for hiding this comment

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

Hi, thanks very much for fixing the error. Could you fix the indent please. nvm, your indent is fine, tabs are used on subsequent lines causing them to be expanded into multiple spaces on Github.

@gnmearacaun gnmearacaun merged commit 229ade0 into LunarVim:06-LSP Apr 5, 2023
@trekie86
Copy link
Author

trekie86 commented Apr 5, 2023

Great thanks. I thought the spacing was accurate but I'll double check my nvim config to ensure it's replacing tabs with spaces to keep things consistent.

@trekie86
Copy link
Author

trekie86 commented Apr 5, 2023

This should be able to resolve #294 unless we should go through to address each impacted branch as well.

@gnmearacaun
Copy link
Collaborator

Great thanks. I thought the spacing was accurate but I'll double check my nvim config to ensure it's replacing tabs with spaces to keep things consistent.

For sure, it's configured in options to expand tabs to (2) spaces. Github has a default set to 8 spaces, which is why it appears janky. The repo uses spaces (vs tabs) by default. The default can be changed in the Github user Settings > Appearance to change the default to two spaces, if desired. A few of the files are using tabs due to various authors including parts copied from other repos. There's a handy command :retab to replace tabs with spaces in a file. That's something we ought to do (including your fix above) in master in the fullness of time.

@gnmearacaun
Copy link
Collaborator

This should be able to resolve #294 unless we should go through to address each impacted branch as well.

Thanks, I've closed #294 now.

There are other problems with all of the later branches. Stuff that has been fixed already in master but just needs to be applied to those branches.

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