Skip to content

initial devcontainer implementation#366

Merged
nerdalert merged 2 commits intoinstructlab:mainfrom
Gregory-Pereira:implement-devcontainer
Nov 21, 2024
Merged

initial devcontainer implementation#366
nerdalert merged 2 commits intoinstructlab:mainfrom
Gregory-Pereira:implement-devcontainer

Conversation

@Gregory-Pereira
Copy link
Collaborator

@Gregory-Pereira Gregory-Pereira commented Nov 21, 2024

Addresses: #361

cc @vishnoianil @nerdalert

Everything works like a dream. I motion we publish this image to the quay org. Also if you think of any other cool binaries we could stuff in here that would be great. I was thinking maybe ngrok if they want a free way to emulate HTTPS routes and share one devs build to another

Signed-off-by: greg pereira <grpereir@redhat.com>
@Gregory-Pereira Gregory-Pereira marked this pull request as ready for review November 21, 2024 02:03
@Gregory-Pereira
Copy link
Collaborator Author

This implementation is based off of the https://github.com/devcontainers/images/tree/main/src/typescript-node base image and its parent: https://github.com/devcontainers/images/tree/main/src/javascript-node, both of which have been properly rebased against the official redhat ubi9/nodejs image

@Gregory-Pereira Gregory-Pereira self-assigned this Nov 21, 2024
@Gregory-Pereira Gregory-Pereira added enhancement UX Related to the user experience tech debt CI/CD labels Nov 21, 2024
@Gregory-Pereira Gregory-Pereira force-pushed the implement-devcontainer branch 3 times, most recently from 72e5c00 to 0c17a50 Compare November 21, 2024 15:38
Signed-off-by: greg pereira <grpereir@redhat.com>
brents-pet-robot
brents-pet-robot approved these changes Nov 21, 2024
Copy link
Member

@nerdalert nerdalert left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD tech debt UX Related to the user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants