Skip to content

Fix RenameFiles workspace operations - #1079

Merged
JoeRobich merged 2 commits into
OmniSharp:masterfrom
manandre:file-rename
Sep 13, 2024
Merged

JoeRobich merged 2 commits into
OmniSharp:masterfrom
manandre:file-rename

Conversation

@manandre

@manandre manandre commented Oct 22, 2023 •

Copy link
Copy Markdown
Contributor

Fixes #1078

@github-actions github-actions Bot added this to the v0.19.10 milestone Oct 22, 2023

await client.RequestWillCreateFile(
new WillCreateFileParams
await client.RequestWillCreateFiles(

@manandre manandre Oct 22, 2023 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The plural form seems to impact the ILanguageClient extension methods. Is it considered as a breaking change?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we are fine to take the changes, but I would like us to start a breaking changes document to catalogue the breaking changes in each version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mysterious We forgot to label this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FileRename record implementation is incorrect

2 participants