Skip to content

Properly set notebook state based on cell execution - #571

Open
SanjulaGanepola wants to merge 1 commit into
mainfrom
fix/notebook-state
Open

Properly set notebook state based on cell execution#571
SanjulaGanepola wants to merge 1 commit into
mainfrom
fix/notebook-state

Conversation

@SanjulaGanepola

Copy link
Copy Markdown
Member

Changes

This PR now properly sets the cell statement based on the execution state (applies to CL, SQL, and shell scripts)

image

How to test this PR

  1. Create a notebook and run some invalid SQL/CL and observe the error icon.
  2. Repeat above with valid SQL/CL.

Checklist

  • have tested my change

Signed-off-by: Sanjula Ganepola <Sanjula.Ganepola@ibm.com>
@github-actions

Copy link
Copy Markdown
Contributor

👋 A new build is available for this PR based on ef72701.

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.

1 participant