Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Set Correct Payload for Dataflow Errors #1484

Merged
merged 2 commits into from
Feb 18, 2021
Merged

Conversation

4e6
Copy link
Contributor

@4e6 4e6 commented Feb 17, 2021

Pull Request Description

close #1480

Set DataflowError payload for dataflow error updates.

Important Notes

Checklist

Please include the following checklist in your PR:

  • The documentation has been updated if necessary.
  • All code conforms to the Scala, Java, and Rust style guides.
  • All documentation and configuration conforms to the markdown and YAML style guides.
  • All code has been tested where possible.

@4e6 4e6 added Type: Bug p-highest Should be completed ASAP labels Feb 17, 2021
@4e6 4e6 requested a review from iamrecursion as a code owner February 17, 2021 07:59
@4e6 4e6 self-assigned this Feb 17, 2021
@4e6 4e6 requested review from kustosz and radeusgd as code owners February 17, 2021 07:59
@4e6 4e6 merged commit ad4d628 into main Feb 18, 2021
@4e6 4e6 deleted the wip/db/dataflow-error-payload branch February 18, 2021 12:05
iamrecursion pushed a commit that referenced this pull request Feb 23, 2021
Set DataflowError payload for dataflow error updates.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
p-highest Should be completed ASAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Language Server is not always reporting errors correctly.
2 participants