Skip to content

docs: Add a section in the Bun guide to use DHI #23565

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Oct 17, 2025
Merged

docs: Add a section in the Bun guide to use DHI #23565

merged 10 commits into from
Oct 17, 2025

Conversation

@Pradumnasaraf
Copy link
Contributor

@Pradumnasaraf Pradumnasaraf commented Oct 15, 2025

Description

Add a section in the Bun guide to use DHI

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify
Copy link

netlify bot commented Oct 15, 2025 •
edited
Loading

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit f5668d4
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/68f2357c8881b40007de3b12
😎 Deploy Preview https://deploy-preview-23565--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Pradumnasaraf Pradumnasaraf marked this pull request as draft October 15, 2025 17:43
@Pradumnasaraf Pradumnasaraf marked this pull request as ready for review October 15, 2025 18:08
usha-mandya
usha-mandya reviewed Oct 15, 2025
View reviewed changes
usha-mandya
usha-mandya reviewed Oct 15, 2025
View reviewed changes
usha-mandya
usha-mandya reviewed Oct 15, 2025
View reviewed changes
@Pradumnasaraf
Copy link
Contributor Author

Pradumnasaraf commented Oct 15, 2025

Hey, @usha-mandya. Great call. I have updated the docs. Thanks for the review!

craig-osterhout
craig-osterhout requested changes Oct 15, 2025
View reviewed changes
Copy link
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

Thanks @Pradumnasaraf. One small issue noted. Everything else appears to work smoothly.

@Pradumnasaraf
Copy link
Contributor Author

Pradumnasaraf commented Oct 16, 2025

Ah, good spot, @craig-osterhout. Yes, we can't have lateston DHI. I think the auto-completion tool edited that. I have fixed it. Thanks!

@usha-mandya
Copy link
Member

usha-mandya commented Oct 16, 2025

@Pradumnasaraf The updates aren't displayed in the deploy preview - https://deploy-preview-23565--docsdocker.netlify.app/guides/bun/. Not sure if I'm missing something. Could you PTAL?

@Pradumnasaraf
Copy link
Contributor Author

Pradumnasaraf commented Oct 16, 2025

Hey @usha-mandya, for some reason, the preview did not get triggered. I have combined all the commits and pushed it (to trigger the preview again). Now you will be able to see it https://deploy-preview-23565--docsdocker.netlify.app/guides/bun/containerize/

usha-mandya
usha-mandya requested changes Oct 16, 2025
View reviewed changes
Pradumnasaraf and others added 7 commits October 16, 2025 23:34
@Pradumnasaraf @usha-mandya
Update content/guides/bun/containerize.md
122cff7 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
@Pradumnasaraf @usha-mandya
Update content/guides/bun/containerize.md
e9c7cac 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
@Pradumnasaraf @usha-mandya
Update content/guides/bun/containerize.md
31074a4 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
@Pradumnasaraf @usha-mandya
Update content/guides/bun/containerize.md
9a5d747 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
@Pradumnasaraf @usha-mandya
Update content/guides/bun/containerize.md
6d78ccf 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
@Pradumnasaraf @usha-mandya
Update content/guides/bun/containerize.md
2d3b884 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
@Pradumnasaraf @usha-mandya
Update content/guides/bun/containerize.md
35a3241 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
usha-mandya
usha-mandya reviewed Oct 17, 2025
View reviewed changes
@Pradumnasaraf @usha-mandya
Update content/guides/bun/_index.md
4ef49bd 
Co-authored-by: Usha Mandya <47779042+usha-mandya@users.noreply.github.com>
@Pradumnasaraf
Update Bun getting started guide description
f5668d4 
Removed mention of Docker Hardened Images (DHI) from the guide description.
craig-osterhout
craig-osterhout approved these changes Oct 17, 2025
View reviewed changes
Copy link
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

LGTM

@craig-osterhout craig-osterhout merged commit c0e404f into docker:main Oct 17, 2025
15 checks passed
@edithturn edithturn mentioned this pull request Nov 16, 2025
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@usha-mandya usha-mandya usha-mandya approved these changes

@craig-osterhout craig-osterhout craig-osterhout approved these changes

Assignees

No one assigned

Labels

area/guides

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Pradumnasaraf @usha-mandya @craig-osterhout