Skip to content

[python] - solution to features issue #870 - unable to open codespace in jupyterlab - #1156

Closed
gauravsaini04 wants to merge 4 commits into
devcontainers:mainfrom
gauravsaini04:codespace_notopen_in_jupyterlab_issue_#870
Closed

[python] - solution to features issue #870 - unable to open codespace in jupyterlab#1156
gauravsaini04 wants to merge 4 commits into
devcontainers:mainfrom
gauravsaini04:codespace_notopen_in_jupyterlab_issue_#870

Conversation

@gauravsaini04

Copy link
Copy Markdown
Contributor

Ref: #870

Feature Name

  • Python

Description

Changelog

  • Have added to containerEnv['PATH'] in devcontainer-feature.json in python feature

Checklist

  • Checked that applied changes work as expected

"PIPX_HOME": "/usr/local/py-utils",
"PIPX_BIN_DIR": "/usr/local/py-utils/bin",
"PATH": "/usr/local/python/current/bin:/usr/local/py-utils/bin:${PATH}"
"PATH": "/usr/local/python/current/bin:/usr/local/py-utils/bin:/usr/local/share/nvm/current/bin:/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/home/vscode/.local/bin:${PATH}"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The user name vscode is hard coded. Is that ideal?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1, This solution would prevent the feature from working for people who have specified the containerUser

@gauravsaini04
gauravsaini04 marked this pull request as ready for review October 24, 2024 00:04
@gauravsaini04
gauravsaini04 requested a review from a team as a code owner October 24, 2024 00:04
@Mathiyarasy

Copy link
Copy Markdown
Contributor

Closing this PR The issue is addressed in PR #1199

@Mathiyarasy Mathiyarasy closed this Dec 5, 2024
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.

4 participants