From b72c54fc311c7bd56f6aab2eb6212a2d43a51afe Mon Sep 17 00:00:00 2001 From: Marshall Bowers Date: Wed, 6 Dec 2023 16:59:24 -0500 Subject: [PATCH] Don't load the Vim keymap temporarily (#3522) This PR removes the loading of the Vim keymap temporarily. This cuts down on the noise from all of the Vim-related action warnings. We can resume loading the Vim keymap once we're ready to add Vim support. Release Notes: - N/A --- crates/settings2/src/settings_file.rs | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/crates/settings2/src/settings_file.rs b/crates/settings2/src/settings_file.rs index 590079c51b..46450a9c28 100644 --- a/crates/settings2/src/settings_file.rs +++ b/crates/settings2/src/settings_file.rs @@ -124,6 +124,17 @@ pub fn update_settings_file( pub fn load_default_keymap(cx: &mut AppContext) { for path in ["keymaps/default.json", "keymaps/vim.json"] { + // TODO: Remove this conditional when we're ready to add Vim support. + // Right now we're avoiding loading the Vim keymap to silence the warnings + // about invalid action bindings. + if path.contains("vim") { + let _: Option<()> = Err(format!( + "TODO: Skipping {path} until we're ready to add Vim support" + )) + .log_err(); + continue; + } + KeymapFile::load_asset(path, cx).unwrap(); }