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

Error body parsing #480

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

atesgoral
Copy link
Contributor

@atesgoral atesgoral commented May 14, 2024

Add minimal reproducible test case for #479

CI should fail.

I'll try to follow up with a fix.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?

@alexrudall
Copy link
Owner

Thanks very much for this, @atesgoral - did you have an idea for the fix here?

@atesgoral
Copy link
Contributor Author

@alexrudall I started looking at a fix a while back, but it didn't turn out trivial due to all the streaming / passing around of blocks. I don't remember the details. But this is something I think is still worth patching!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants