Skip to content

Remove website tooling from build-image (#7342)#7345

Merged
friedrichg merged 2 commits intomasterfrom
remove-website-from-build-image
Mar 11, 2026
Merged

Remove website tooling from build-image (#7342)#7345
friedrichg merged 2 commits intomasterfrom
remove-website-from-build-image

Conversation

@friedrichg
Copy link
Member

Phase 3: Remove Node.js, Hugo, postcss-cli, autoprefixer, and embedmd from the build-image Dockerfile. Website build and deploy are now handled entirely by .github/workflows/web.yml.

Also removes the website build steps and disabled deploy_website job from test-build-deploy.yml.

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@dosubot dosubot bot added the ci/cd label Mar 10, 2026
Phase 3: Remove Node.js, Hugo, postcss-cli, autoprefixer, and embedmd
from the build-image Dockerfile. Website build and deploy are now
handled entirely by .github/workflows/web.yml.

Also removes the website build steps and disabled deploy_website job
from test-build-deploy.yml.

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg force-pushed the remove-website-from-build-image branch from d6cc5a8 to 70218e6 Compare March 10, 2026 06:19
@SungJin1212
Copy link
Member

Should we delete unused targets in the Makefile, like web-pre, web-build, and web-deploy?

For upgrading docsy, we can just use the hugo command.
I will do that i an separate PR

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 11, 2026
@friedrichg friedrichg merged commit 0fd493f into master Mar 11, 2026
64 of 67 checks passed
@friedrichg friedrichg deleted the remove-website-from-build-image branch March 11, 2026 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd lgtm This PR has been approved by a maintainer size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants