Skip to content

Address review comments from #1322 and #1329 #1349

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 2 commits into from
May 29, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented May 24, 2024

No description provided.

@ahoppen ahoppen requested review from bnbarham and hamishknight May 24, 2024 19:00
@ahoppen ahoppen requested a review from benlangmuir as a code owner May 24, 2024 19:00
@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test

Show which index tasks are currently running in the indexing work done progress. \
This produces a multi-line work done progress, which might render incorrectly depending in the editor.
When reporting index progress, show the currently running index tasks in addition to the task's count. \
This produces a multi-line work done progress, which might render incorrectly, depending in the editor.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This produces a multi-line work done progress, which might render incorrectly, depending in the editor.
This produces a multi-line work done progress, which might render incorrectly, depending on the editor.

@ahoppen ahoppen force-pushed the address-review-comments-1322-1329 branch from 7b79431 to 8810dbe Compare May 24, 2024 19:35
@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test Windows

2 similar comments
@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test Windows

@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test Windows

@ahoppen ahoppen force-pushed the address-review-comments-1322-1329 branch from 8810dbe to 8772ccf Compare May 25, 2024 00:21
@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test Windows

@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test Linux

@ahoppen ahoppen force-pushed the address-review-comments-1322-1329 branch from 8772ccf to 9098e09 Compare May 25, 2024 20:42
@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test Windows

@ahoppen
Copy link
Member Author

ahoppen commented May 27, 2024

@swift-ci Please test Linux

1 similar comment
@ahoppen
Copy link
Member Author

ahoppen commented May 28, 2024

@swift-ci Please test Linux

@ahoppen
Copy link
Member Author

ahoppen commented May 28, 2024

@swift-ci Please test Linxu

@ahoppen
Copy link
Member Author

ahoppen commented May 28, 2024

@swift-ci Please test Linux

@ahoppen ahoppen merged commit 0f86e01 into swiftlang:main May 29, 2024
3 checks passed
@ahoppen ahoppen deleted the address-review-comments-1322-1329 branch May 29, 2024 00:30
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.

3 participants