Skip to content

Make LSPDocumentDiagnostic close location list if no error.#1543

Open
tshirtman wants to merge 1 commit into
prabirshrestha:masterfrom
tshirtman:close-location-list-if-no-error
Open

Make LSPDocumentDiagnostic close location list if no error.#1543
tshirtman wants to merge 1 commit into
prabirshrestha:masterfrom
tshirtman:close-location-list-if-no-error

Conversation

@tshirtman

Copy link
Copy Markdown

No description provided.

@tshirtman tshirtman left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fix #817

@stale

stale Bot commented Apr 26, 2025

Copy link
Copy Markdown

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the wontfix label Apr 26, 2025
@barreyra

Copy link
Copy Markdown

Keep alive comment.

@stale stale Bot removed the wontfix label Apr 28, 2025
@stale

stale Bot commented Jul 19, 2025

Copy link
Copy Markdown

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the wontfix label Jul 19, 2025
@barreyra

Copy link
Copy Markdown

Keep alive comment.

@stale stale Bot removed the wontfix label Jul 19, 2025
@mattn

mattn commented Jan 1, 2026

Copy link
Copy Markdown
Collaborator

Please add new option to behave this.

@tshirtman

Copy link
Copy Markdown
Author

Please add new option to behave this.

I guess that behavior can be made optional, yes, the original issue was closed as wontfix, will this be merged if the option is added?

@tshirtman tshirtman force-pushed the close-location-list-if-no-error branch from 9b9b9b5 to 503ff93 Compare January 3, 2026 14:24
@mattn

mattn commented Mar 7, 2026

Copy link
Copy Markdown
Collaborator

will this be merged if the option is added?

Yes.

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