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 multi-stage builds to the C++ guide #21562

Merged
merged 5 commits into from
Feb 27, 2025

Conversation

aerabi
Copy link
Contributor

@aerabi aerabi commented Dec 4, 2024

Description

Having a multi-stage build is an essential thing to learn for containerizing a compiled language.

Related issues or tickets

N/A

Reviews

  • Technical review
  • Editorial review
  • Product review

Copy link

netlify bot commented Dec 4, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit d358ae0
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/67bf8ac9ca555e00082b47a3
😎 Deploy Preview https://deploy-preview-21562--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 site configuration.

@aerabi aerabi force-pushed the aerabi/add-multistage branch from 63107cf to ff86f7f Compare February 25, 2025 22:25
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.

Thank you. It looks good. There are couple minor fixes.
Also, can you help update the clone repository command in develop.md from git clone https://github.com/Pradumnasaraf/c-plus-plus-docker.git to git clone https://github.com/dockersamples/c-plus-plus-docker.git

@aerabi
Copy link
Contributor Author

aerabi commented Feb 26, 2025

Thanks for the reviews, @craig-osterhout! I pushed another commit, PTAL. 🙏
I'll also update Pradumna's repo separately.

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.

Thank you. LGTM.

@craig-osterhout craig-osterhout merged commit 25d4796 into docker:main Feb 27, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants