✅ Deploy Preview for docsdocker ready!

Name	Link
🔨 Latest commit	0a5341b
🔍 Latest deploy log	https://app.netlify.com/projects/docsdocker/deploys/684c66f94173880008f93bb9
😎 Deploy Preview	https://deploy-preview-22777--docsdocker.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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

@kasunsjc , @brandonh6k , @HofmeisterAn , @jmaitrehenry, @igor-alexandrov

Could you please review the code again when you get a chance?
I’d appreciate a second look to ensure everything aligns with the expectations. Thanks!

condition: and(succeeded(), eq(variables['Build.SourceBranch'], 'refs/heads/main'))
We trigger the pipeline for PR too, you may want to move the condition in the Push Docker Image task instead.

Also update the #### Step 4.3 Build the Docker Image with the sbom and attestation. Maybe we could reference the SBOM and attestion doc page:
https://docs.docker.com/build/metadata/attestations/slsa-provenance/#:~:text=To%20create%20a%20provenance%20attestation,of%20%2D%2Dattest%20type%3Dprovenance%20.
https://docs.docker.com/build/ci/github-actions/attestations/#:~:text=To%20add%20SBOM%20attestations%2C%20set,image%20store%20of%20the%20runner.

Could you please review the code again when you get a chance?
I’d appreciate a second look to ensure everything aligns with the expectations. Thanks!

@kristiyan-velkov I checked the PR again. It definitely became better, but the thing I originally pointed to – the unneeded stages: key is still there. What did you decide about it?

@kristiyan-velkov good job! I left a small comment to simplify the guide for potential readers.

Furthermore, in the GHA Guide, instead of providing the complete action at the beginning of the guide, David posted parts of it explaining in details what each of them does. In my opinion, his approach is better considering readers' knowledge level.

What do you think?

Done!

condition: and(succeeded(), eq(variables['Build.SourceBranch'], 'refs/heads/main')) We trigger the pipeline for PR too, you may want to move the condition in the Push Docker Image task instead.

Also update the #### Step 4.3 Build the Docker Image with the sbom and attestation. Maybe we could reference the SBOM and attestion doc page: https://docs.docker.com/build/metadata/attestations/slsa-provenance/#:~:text=To%20create%20a%20provenance%20attestation,of%20%2D%2Dattest%20type%3Dprovenance%20. https://docs.docker.com/build/ci/github-actions/attestations/#:~:text=To%20add%20SBOM%20attestations%2C%20set,image%20store%20of%20the%20runner.

Done!

Could you please review the code again when you get a chance?
I’d appreciate a second look to ensure everything aligns with the expectations. Thanks!

@kristiyan-velkov I checked the PR again. It definitely became better, but the thing I originally pointed to – the unneeded stages: key is still there. What did you decide about it?

Thanks for checking again!
The stages: key is still there because if someone plans to expand the pipeline later—potentially adding stages for testing, security checks, or deployments.
While it's not required for a single-stage setup, keeping it now avoids a bigger restructure down the line.

Hey @sarahsanders-docker ,

I’ve completed all the updates and incorporated your feedback carefully. Thank you for taking the time to review and provide such detailed suggestions—it really helps keep the guide clear, professional, and aligned with our style standards. If anything else comes up or you notice more tweaks, don’t hesitate to reach out.

I appreciate your support and collaboration on this! 🚀