Skip to content

[clangd] Add an indexedRename request #7973

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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 21, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Jan 13, 2024

This request can be used by sourcekit-lsp to get the edits necessary to rename a symbol whose occurrences are known from the IndexStore that is kept by sourcekit-lsp.

@ahoppen ahoppen requested a review from bnbarham January 13, 2024 01:53
@ahoppen
Copy link
Member Author

ahoppen commented Jan 13, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Jan 13, 2024

@swift-ci Please test llvm

1 similar comment
@ahoppen
Copy link
Member Author

ahoppen commented Jan 18, 2024

@swift-ci Please test llvm

@ahoppen
Copy link
Member Author

ahoppen commented Jan 18, 2024

All test failures are unrelated

This request can be used by sourcekit-lsp to get the edits necessary to rename a symbol whose occurrences are known from the IndexStore that is kept by sourcekit-lsp.
@ahoppen ahoppen force-pushed the ahoppen/indexed-rename branch from 059b7d5 to 39ca137 Compare January 19, 2024 17:30
@ahoppen
Copy link
Member Author

ahoppen commented Jan 19, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Jan 19, 2024

@swift-ci Please test llvm

@ahoppen
Copy link
Member Author

ahoppen commented Jan 20, 2024

@swift-ci Please test Windows

@ahoppen
Copy link
Member Author

ahoppen commented Jan 21, 2024

LLVM test failures are unrelated.

@ahoppen ahoppen merged commit 5f9f233 into swiftlang:stable/20230725 Jan 21, 2024
@ahoppen ahoppen deleted the ahoppen/indexed-rename branch January 21, 2024 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants