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

add dev tools to devcontainer build #210

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

pstjohn
Copy link
Collaborator

@pstjohn pstjohn commented Sep 28, 2024

Adds development tools from requirements-dev.txt to the devcontainer image.

This also reverts #123 since it seems to have introduced some import errors

@pstjohn pstjohn force-pushed the pstjohn/main/devcontainer-devtools branch from 4b9cbec to 6b78e03 Compare September 30, 2024 17:29
@pstjohn pstjohn force-pushed the pstjohn/main/devcontainer-devtools branch from 6b78e03 to d53afdf Compare September 30, 2024 17:45
@pstjohn pstjohn changed the title add dev tools to devcontainer build add dev tools to devcontainer build and revert lightning module change Sep 30, 2024
@pstjohn
Copy link
Collaborator Author

pstjohn commented Sep 30, 2024

/build-ci

@pstjohn pstjohn force-pushed the pstjohn/main/devcontainer-devtools branch from d53afdf to 262cc98 Compare September 30, 2024 17:57
@pstjohn pstjohn changed the title add dev tools to devcontainer build and revert lightning module change add dev tools to devcontainer build Sep 30, 2024
@pstjohn pstjohn enabled auto-merge (squash) September 30, 2024 18:13
@pstjohn pstjohn disabled auto-merge September 30, 2024 18:13
@pstjohn pstjohn force-pushed the pstjohn/main/devcontainer-devtools branch from 83fe18b to 0c28961 Compare September 30, 2024 18:42
@pstjohn
Copy link
Collaborator Author

pstjohn commented Sep 30, 2024

/build-ci

@pstjohn pstjohn enabled auto-merge (squash) September 30, 2024 18:47
@pstjohn pstjohn merged commit e53a255 into NVIDIA:main Sep 30, 2024
3 checks passed
tshimko-nv pushed a commit that referenced this pull request Oct 2, 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