Skip to content

Conversation

@goanpeca
Copy link
Collaborator

@goanpeca goanpeca commented Oct 17, 2025

Needed for datalayer/vscode-datalayer#31

Ref image
Screenshot 2025-10-17 at 12 39 45 PM

@goanpeca goanpeca self-assigned this Oct 17, 2025
@goanpeca goanpeca marked this pull request as ready for review October 17, 2025 17:50
@goanpeca goanpeca requested review from Copilot and echarles October 17, 2025 17:50
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds new Lexical commands to control Jupyter cells from the editor: run the current cell, run all cells, restart the kernel, and clear all outputs. Registers handlers for each command within the JupyterInputOutputPlugin.

  • Adds four new commands: RUN_JUPYTER_CELL_COMMAND, RUN_ALL_JUPYTER_CELLS_COMMAND, RESTART_JUPYTER_KERNEL_COMMAND, CLEAR_ALL_OUTPUTS_COMMAND
  • Implements command handlers to execute the focused cell, execute all cells, restart the kernel, and clear outputs
  • Logs status and error information during command execution

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@goanpeca goanpeca marked this pull request as draft October 17, 2025 18:01
@goanpeca goanpeca force-pushed the enh/add-jupyter-lexical-commnands branch from 0eed580 to ffede9d Compare October 17, 2025 18:29
@goanpeca goanpeca force-pushed the enh/add-jupyter-lexical-commnands branch from ffede9d to adc6ed5 Compare October 17, 2025 19:36
@goanpeca goanpeca marked this pull request as ready for review October 17, 2025 20:03
Copy link
Member

@echarles echarles left a comment

Choose a reason for hiding this comment

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

LGTM Thx @goanpeca

@echarles echarles merged commit e819e6c into datalayer:main Oct 18, 2025
6 checks passed
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.

2 participants