fix(Handlers.Rename): Do not rename in string or commends, restore fi…

…le rename As noted by @Myrslaver in #157
razzmatazz · May 21, 2024 · ccee0ef · ccee0ef
1 parent 2bdcec7
commit ccee0ef
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,8 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ## [Unreleased]
+* Do not rename symbol in strings or comments, restore file rename
+  - As reported by @Myrslaver in https://github.com/razzmatazz/csharp-language-server/issues/157
 * Stop collecting symbols in method body
   - By Adam Tao @tcx4c70
 * Fix TypeHierarchy registration

diff --git a/src/CSharpLanguageServer/Handlers/Rename.fs b/src/CSharpLanguageServer/Handlers/Rename.fs
@@ -167,7 +167,7 @@ module Rename =
                 Renamer.RenameSymbolAsync(
                     doc.Project.Solution,
                     symbol,
-                    SymbolRenameOptions(RenameOverloads = true, RenameInStrings = true, RenameInComments = true),
+                    SymbolRenameOptions(RenameOverloads = true, RenameFile = true),
                     p.NewName,
                     ct
                 )