Skip to content

When a sourcekitd diagnostics request fails, show the request error as a diagnostic on the source file #1886

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
Dec 13, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Dec 13, 2024

The missing diagnostics might be due to an error that the user can fix. Report it to them.

Fixes #1812
rdar://139514623

…s a diagnostic on the source file

The missing diagnostics might be due to an error that the user can fix. Report it to them.

Fixes swiftlang#1812
rdar://139514623
@ahoppen ahoppen requested a review from bnbarham December 13, 2024 00:32
@ahoppen
Copy link
Member Author

ahoppen commented Dec 13, 2024

@swift-ci Please test

Copy link
Contributor

@bnbarham bnbarham left a comment

Choose a reason for hiding this comment

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

Very interested to know whether you end up finding this annoying or not 😅. Is it worth having an option to turn it off, or will we just drop this if the general feedback is that it isn't helpful?

@ahoppen
Copy link
Member Author

ahoppen commented Dec 13, 2024

I’d say let’s just enable this by default for now. If we discover that it’s more distracting than helpful, we would probably just revert this PR again.

@ahoppen ahoppen merged commit 9771138 into swiftlang:main Dec 13, 2024
3 checks passed
@ahoppen ahoppen deleted the sourcekitd-request-diagnostics branch December 13, 2024 18:42
ahoppen added a commit to ahoppen/sourcekit-lsp that referenced this pull request Dec 17, 2024
…uest-diagnostics"

This reverts commit 9771138, reversing
changes made to 9a1c75e.
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.

sourcekitd silently fails
2 participants