Skip to content

Revert "release: merge upstream sub-site on deploy"#24493

Merged
dvdksn merged 1 commit intomainfrom
revert-24475-deploy-ds
Mar 24, 2026
Merged

Revert "release: merge upstream sub-site on deploy"#24493
dvdksn merged 1 commit intomainfrom
revert-24475-deploy-ds

Conversation

@dvdksn
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn commented Mar 24, 2026

Reverts #24475

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 24, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 76777ff
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69c24cf8c9b0860008911ac4
😎 Deploy Preview https://deploy-preview-24493--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.

@github-actions github-actions bot added area/release Relates to CI or deployment ci ci updates labels Mar 24, 2026
@dvdksn dvdksn requested a review from usha-mandya March 24, 2026 08:38
Copy link
Copy Markdown
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This is a clean revert of PR #24475. The changes remove:

  • Docker Hub login step
  • Merge sub-site step (which extracted content from docs/dash:dist-latest)

The revert is complete with no leftover references. The workflow will continue to function correctly as the subsequent steps only depend on the public directory, which is still produced by the existing build step.

No issues detected.

@dvdksn dvdksn merged commit dfa8f0f into main Mar 24, 2026
19 checks passed
@dvdksn dvdksn deleted the revert-24475-deploy-ds branch March 24, 2026 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/release Relates to CI or deployment ci ci updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants