Skip to content

Conversation

@ahoppen
Copy link
Member

@ahoppen ahoppen commented May 25, 2024

While SemanticIndexManager.inProgressPrepareForEditorTask is not nil, show a work done progress in the editor that the current file is being prepared for editor functionality.

I decided to use the indexing progress indicator for this for now because I think it’s too spammy if SourceKit-LSP has two different progress indicators for preparation and indexing. I’ll need to see how this feels like in practice.

rdar://128722609

…tor functionality

While `SemanticIndexManager.inProgressPrepareForEditorTask` is not `nil`, show a work done progress in the editor that the current file is being prepared for editor functionality.

I decided to use the indexing progress indicator for this for now because I think it’s too spammy if SourceKit-LSP has two different progress indicators for preparation and indexing. I’ll need to see how this feels like in practice.

rdar://128722609
@ahoppen ahoppen requested review from bnbarham and hamishknight May 25, 2024 17:41
@ahoppen ahoppen requested a review from benlangmuir as a code owner May 25, 2024 17:41
@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test

@ahoppen ahoppen merged commit 24dfbd5 into swiftlang:main May 28, 2024
@ahoppen ahoppen deleted the progress-indicator-current-file-preparation branch May 29, 2024 16:57
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