diff --git a/helix-core/src/diagnostic.rs b/helix-core/src/diagnostic.rs
index f8d92b17b86f6..28686a311b8a0 100644
--- a/helix-core/src/diagnostic.rs
+++ b/helix-core/src/diagnostic.rs
@@ -58,8 +58,8 @@ pub struct Diagnostic {
 // TODO turn this into an enum + feature flag when lsp becomes optional
 pub type DiagnosticProvider = LanguageServerId;
 
-// while I would prefe having this in helix-lsp that necessitates a bucnh of
-// conversions I would rather not add I think its file since this just a very
+// while I would prefer having this in helix-lsp that necessitates a bunch of
+// conversions I would rather not add. I think its fine since this just a very
 // trivial newtype wrapper and we would need something similar once we define
 // completions in core
 slotmap::new_key_type! {