Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add the bun ecosystem #1411

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

Merged
merged 1 commit into from
Feb 21, 2025
Merged

Add the bun ecosystem #1411

merged 1 commit into from
Feb 21, 2025

Conversation

markhallen
Copy link
Contributor

@markhallen markhallen commented Feb 21, 2025

We are adding the bun ecosystem images to the project.

Part of https://github.com/github/dependabot-updates/issues/7990

After updating docker/containers.json and src/update-containers.ts, and then adding docker/Dockerfile.bun, we ran npm run-script package.

The npm command could be found in the package.json scripts here:

"package": "ncc build -o dist/main src/main.ts --source-map --license licenses.txt && ncc build -o dist/cleanup src/cleanup.ts --source-map --license licenses.txt",

@markhallen markhallen marked this pull request as ready for review February 21, 2025 14:44
@Copilot Copilot AI review requested due to automatic review settings February 21, 2025 14:44
@markhallen markhallen requested a review from a team as a code owner February 21, 2025 14:44
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This PR adds support for the Bun ecosystem by including a new Dockerfile and wiring it into the container image update process.

  • Adds a new "bun" entry to the container manifest in update-containers.ts.
  • Introduces support for processing Dockerfile.bun.

Reviewed Changes

File Description
src/update-containers.ts Added new entry "bun" to support the Bun ecosystem by retrieving the image name from Dockerfile.bun

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@markhallen markhallen enabled auto-merge February 21, 2025 14:58
Copy link

@randhircs randhircs left a comment

Choose a reason for hiding this comment

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

I believe you have added configs to all the respective places, I do not have full clarity on this. I approve it.

@markhallen markhallen merged commit 2f44ce8 into main Feb 21, 2025
9 checks passed
@markhallen markhallen deleted the markhallen/adding-the-bun-ecosystem branch February 21, 2025 15:09
@honeyankit honeyankit mentioned this pull request Mar 10, 2025
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.

2 participants