Skip to content

Conversation

@gauravsaini04
Copy link
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
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
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
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