starlark_lsp: remove some autocomplete trigger chars

Summary:
Autocomplete after these characters is more annoying than useful.
Ideally this should be configurable during LSP initialization instead of hardcoded.

Reviewed By: ndmitchell

Differential Revision: D53530861

fbshipit-source-id: 772102fe82c8fbea48a0c5d3d934c3999457bd5f

starlark_lsp/src/server.rs

@@ -410,20 +410,10 @@ impl<T: LspContext> Backend<T> {
                 trigger_characters: Some(vec![
                     // e.g. function call
                     "(".to_owned(),
-                    // e.g. list creation, function call
-                    ",".to_owned(),
-                    // e.g. when typing a load path
-                    "/".to_owned(),
-                    // e.g. dict creation
-                    ":".to_owned(),
                     // e.g. variable assignment
                     "=".to_owned(),
-                    // e.g. list creation
-                    "[".to_owned(),
                     // e.g. string literal (load path, target name)
                     "\"".to_owned(),
-                    // don't lose autocomplete when typing a space, e.g. after a comma
-                    " ".to_owned(),
                 ]),
                 ..Default::default()
             }),