Skip to content

[6.0] Add a request to re-index all files in SourceKit-LSP #1526

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
Jun 27, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Jun 27, 2024

  • Explanation: Users should not need to rely on this request. The index should always be updated automatically in the background. Having to invoke this request manes there is a bug in SourceKit-LSP's automatic re-indexing. It does, however, offer a workaround to re-index files when such a bug occurs where otherwise there would be no workaround.
  • Scope: A new request, purely additive
  • Risk: Low, purely additive
  • Testing: Added test case
  • Issue: rdar://127476221, Manual re-indexing #1263
  • Reviewer: @bnbarham and @hamishknight on Add a request to re-index all files in SourceKit-LSP #1507

Users should not need to rely on this request. The index should always be updated automatically in the background. Having to invoke this request manes there is a bug in SourceKit-LSP's automatic re-indexing. It does, however, offer a workaround to re-index files when such a bug occurs where otherwise there would be no workaround.

rdar://127476221
Resolves swiftlang#1263
@ahoppen ahoppen requested a review from bnbarham June 27, 2024 03:16
@ahoppen
Copy link
Member Author

ahoppen commented Jun 27, 2024

@swift-ci Please test

@ahoppen ahoppen merged commit 0f44f03 into swiftlang:release/6.0 Jun 27, 2024
3 checks passed
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