Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

Add bun devcontainer #1526

Closed
wants to merge 1 commit into from
Closed

Add bun devcontainer #1526

wants to merge 1 commit into from

Conversation

bherbruck
Copy link

No description provided.

@ghost
Copy link

ghost commented Jul 11, 2022

CLA assistant check
All CLA requirements met.

@bamurtaugh
Copy link
Member

Thanks so much for your contribution, @bherbruck!

We're actively working on how to best support current and future dev container contributions. As this poses a delay in accepting new definitions, we want to emphasize we really appreciate your work, and we're working on how to best support fantastic community contributions like yours in a scalable way moving forward. There are additional details in this issue: #1291.


ENV BUN_INSTALL=/bun
RUN mkdir -p /bun \
&& curl -fsSL https://bun.sh/install | sh \

Choose a reason for hiding this comment

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

I think sh would start to fail since bun.sh changed the curl command to bash.
Refer the working devcontainer from @alertbox/vscode-remote-try-bun

@bamurtaugh
Copy link
Member

Thanks again for opening @bherbruck!

As a heads up, our team has been actively focused on an updated plan for community contributions and this repo moving forward, which we've now outlined in this issue: #1589. This includes moving to a couple new repos for images (https://github.com/devcontainers/images) and Features (https://github.com/devcontainers/features).

We anticipate to have a similar repo and distribution process for templates/definitions. We'll keep everyone updated (likely via another issue in this repo or comment on #1589) when our new templates repo is available and the process is defined.

Please let me know if you have any questions, thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants