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

Feature/ed 46 api vscode #928

Merged
merged 7 commits into from
Nov 23, 2023
Merged

Feature/ed 46 api vscode #928

merged 7 commits into from
Nov 23, 2023

Conversation

joncooper65
Copy link
Contributor

This adds VS Code as a Notebook environment.

Key features:

  • it appears as an option when creating a new Notebook in a Project
  • it handles the creation of Conda environments using the same commands as documented for a Jupyter notebook
  • plugins are available in the usual way, especially useful for providing a Python environment
  • Python installed by default

joncooper65 and others added 6 commits November 16, 2023 17:55
…ests all working still and deploymentGenerator.spec.js test extended to include vscode manifest creation test
In a test environment this completes the addition of vscode, although
there are issues with websockets which breaks some of the functionality.

We update all the image_config.json files to include the vscode
additions, so local development and the frontend see the new notebook
type.

Since the token acquisition in stackUrlService.js is identical to
Jupyter notebooks we refactor it into a generic function.

Use of the logo should comply with the restrictions given at
https://code.visualstudio.com/brand.

Jira: ED-45
Jira: ED-46
Copy link
Contributor

@iwalmsley iwalmsley left a comment

Choose a reason for hiding this comment

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

LGTM! Tested it and all working fine if these few things can be fixed.

@iwalmsley iwalmsley merged commit 10242c0 into master Nov 23, 2023
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.

3 participants