Skip to content

Conversation

colombod
Copy link
Member

@colombod colombod commented Nov 2, 2022

fixes #2415

@colombod colombod force-pushed the handle_error_produced branch from 54724c4 to 794b4c3 Compare November 2, 2022 19:06
notify failure to ui


report cell state


test new client behaviour
@colombod colombod force-pushed the handle_error_produced branch from 893af94 to 2ef4b7e Compare November 2, 2022 19:20
@colombod colombod enabled auto-merge (rebase) November 2, 2022 19:59
@@ -108,6 +108,17 @@ value.Display("application/json", "text/html");

#!markdown

# Execute the next cell, the expression should evaluate but the cell fails with the message:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Execute the next cell, the expression should evaluate but the cell fails with the message:
# Execute the next cell. It should fail with the message:

@colombod colombod merged commit 37de37e into dotnet:main Nov 2, 2022
@colombod colombod deleted the handle_error_produced branch November 2, 2022 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

explicitly handle ErrorProduced event type in InteractiveClient.Execute
3 participants