-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
lsp: migrate from lspower
back to tower-lsp
#14056
Comments
(thanks @sno2 for pointing it out) |
is this a good first issue for a new contributor? |
Not entirely sure... It could be very straight forward, but it could get quite tricky if it doesn't work. If someone were to try to do it and the tests didn't pass, it could get quite complicated to figure out, but would be glad to try to help if someone wanted to try. |
Does this have a pre-commit script with which I can locally test the changes? I am up for trying it and I can reach out on discord if I have doubts. Would that work? |
@CGQAQ If I want to add in my changes, can I push it to your PR? |
@AbhijithGanesh Sorry but I am almost done the refactoring, just lack of tests fix 😢 |
No issues , just wanted to know. :) |
lspower
was a fork that fixed a lot of things withtower-lsp
, but notlspower
has been remerged back intotower-lsp
and deprecated.tower-lsp
is being maintained going forward, so we should/need to remigrate back.The text was updated successfully, but these errors were encountered: