Skip to content

Conversation

@Yicong-Huang
Copy link
Contributor

@Yicong-Huang Yicong-Huang commented Oct 17, 2024

This PR fixes:

  1. add a timeout to prevent the language server from connecting forever. currently set to 1 second.
  2. handle the diff editor in the case where no code is found. this could happen when an older version contains an operator A but compared to the latest version that has no A's code (A has been removed).

Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@mengw15 mengw15 left a comment

Choose a reason for hiding this comment

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

LGTM!

@Yicong-Huang Yicong-Huang merged commit 206c953 into master Oct 17, 2024
@Yicong-Huang Yicong-Huang deleted the yicong-fix-diff-editor branch October 17, 2024 06:29
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
This PR fixes:
1. add a timeout to prevent the language server from connecting forever.
currently set to 1 second.
2. handle the diff editor in the case where no code is found. this could
happen when an older version contains an operator A but compared to the
latest version that has no A's code (A has been removed).
@Yicong-Huang Yicong-Huang removed the bug label Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants