New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove older impls #1174
Remove older impls #1174
Conversation
src/FsAutoComplete/Parser.fs
Outdated
@@ -154,7 +154,7 @@ module Parser = | |||
|
|||
let sourceTextFactory: ISourceTextFactory = | |||
match sourceTextFactoryOption with | |||
| SourceTextFactoryOptions.NamedText -> new NamedTextFactory() | |||
| SourceTextFactoryOptions.NamedText -> new RoslynSourceTextFactory() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left in the old parameters for now. Need to test removing these.
yeah, I think so. |
That's fine; let's do it in separate PR, and focus this one on just removing things. |
11a59a0
to
6a29581
Compare
da0df4a
to
dbfe84d
Compare
* Remove NamedText implementation * 馃 Remove FsAutoComplete.Lsp * Fix codefix scaffolding * formatting * Remove unused State/Commands * Cleanup RoslynSourceText * Remove SourceText CLI option
WHAT
馃 Generated by Copilot at 0919866
Removed the
NamedText
type and related code from the project and replaced it with more efficient and consistent source text implementations. Simplified the LSP server implementation and the code fix scaffolding process. Updated the documentation and tests accordingly.馃 Generated by Copilot at 0919866
馃殌馃Ч馃摑
WHY
We've been running on newer implementations for a while and seems to be time to clean up older implementations.
Should I also remove any of the "seams" I put in like
IFSACSourceText
?I also haven't refactored AdaptiveLSP just yet.
HOW
馃 Generated by Copilot at 0919866
NamedText
type and its dependencies from the code base, replacing it with theRoslynSourceText
type that implements theIFSACSourceText
interface more efficiently and consistently (link, link, link, link, link)FsAutoComplete.Lsp.fs
file and theFSharpLspServer
type, and using theAdaptiveFSharpLspServer
type instead (link, link, link, link, link, link, link, link, link)FsAutoComplete.Lsp.fs
file from the registration of the code fix (link, link, link, link, link, link, link)HelpersTests
and theTests
modules in thetest/FsAutoComplete.Tests.Lsp
project to take atextFactory
argument of typeISourceTextFactory
to allow testing different implementations of theIFSACSourceText
interface (link, link, link, link, link, link, link)initNamedText
andaddToNamedText
functions from theHelpers
module in the./benchmarks/SourceTextBenchmarks.fs
file, and the irrelevantNamed_Text_changeText_everyUpdate
benchmark from theSourceText_LineChanges_Benchmarks
type in the same file (link, link, link)