Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

SPL2 Notebooks #90

Merged
merged 44 commits into from
Jul 6, 2023
Merged

SPL2 Notebooks #90

merged 44 commits into from
Jul 6, 2023

Conversation

fantavlik
Copy link
Collaborator

Add support for SPL2 notebooks for customers with access to SPL2 preview deployments.

There is a full wiki page that I've staged to help with setup including some notes for existing SPL Notebooks: https://github.com/fantavlik/vscode-extension-splunk/wiki/Splunk-Notebooks

Features:

  • Guided installation of SPL2 Language Server and Java 17+ (if needed). More details: SPL2 installation #86
  • Ability to execute SPL2 notebooks cells via spl2-module-dispatch endpoint and view job results. More details: Add ability to dispatch SPL2 via SPL2 Notebooks #85
  • Support for SPL2 Language Server and Client initialization when .spl2nb or modules.json files are opened which provides syntax highlighting, autocomplete, hover documentation, code suggestions, and other language server features. More details: Spl2 - Language Server Support #87

fantavlik and others added 30 commits June 2, 2023 13:13
Work still needed to serialize to the correct format.
Add support for notebook commands
Fix job status updates for in-progress jobs
Add better error handling for dispatch responses
Add ability to dispatch SPL2 via SPL2 Notebooks
@fantavlik fantavlik requested a review from JasonConger June 30, 2023 18:25
@fantavlik
Copy link
Collaborator Author

@JasonConger I will be adding tests as a follow-up, we have done testing with 4-5 of our team mostly on Mac and I did Windows testing myself as well all manually.

@JasonConger JasonConger merged commit b5b830d into master Jul 6, 2023
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