-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
KeyError when adding and removing language servers #21161
Comments
Hey @jpx40, thanks for reporting. You said:
Did you add it and then remove it? That's not clear to me from the error you posted above. |
I added it, to use language completion for go. |
I removed a few language servers before i added go. Because the path was wrong. One of the removed language servers was for css. |
Ok, that makes sense because that's what the error is reporting. We'll investigate this more carefully and try to solve it in our next version (5.5.0), to be released in a couple of months. |
Just in case, I was able to reproduce this while checking PR #21215: Traceback (most recent call last):
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\preferences\widgets\configdialog.py", line 142, in button_clicked
configpage.apply_changes()
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\preferences\api.py", line 119, in apply_changes
self.apply_callback()
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\preferences\api.py", line 151, in <lambda>
self._apply_settings_tabs(self.changed_options))
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\preferences\api.py", line 178, in _apply_settings_tabs
options |= widget.apply_settings()
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\completion\plugin.py", line 716, in wrapper
for opt in prev_method(self):
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\completion\providers\languageserver\conftabs\otherlanguages.py", line 104, in apply_settings
return self.table.save_servers()
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\completion\providers\languageserver\widgets\serversconfig.py", line 643, in save_servers
server.delete()
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\completion\providers\languageserver\widgets\serversconfig.py", line 108, in delete
self.remove_option(language)
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\plugins\completion\plugin.py", line 673, in wrapper
return plugin.remove_conf(option, section)
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\api\plugins\new_api.py", line 555, in remove_conf
self._conf.remove_option(section, option)
File "e:\acer\documentos\spyder\spyder otros\carlos\spyder\spyder\config\manager.py", line 572, in remove_option
conf_ptr.pop(last_option)
KeyError: 'bash' Adding a language server and then removing it before clicking |
@dalthviz, please take a look at this one since you were able to reproduce it. |
Description
What steps will reproduce the problem?
I added the go language Server (gopls). And then this error Message showed up. By the way i using MacOS Ventura on an M2 Mac
Traceback
Versions
Dependencies
The text was updated successfully, but these errors were encountered: