Skip to content

Improve remote keyword functionality

Emanuele Roberto Nocera requested to merge betterkeywords into master

Created by: Zaharid

vp-comparefits and other scripts would suggest remote keywords when in interactive mode. The implementation had some problems as it would raise an exception whenever the user had no permission to access vp server which would in turn crash the whole app. The solution there was to issue the appropriate raise_for_status command. This was made worse by the fact that this particular completer would not respect the --no-net flag and try to connect anyway.

While we are on it, tidy up the implementation, moving the bulk of it to RemoteLoader instead. With that make sure that comarefits only looks at that when the remote loader is in use.

cc @J-M-Moore

Merge request reports