ci: optimize GitHub Actions CI/CD workflow #409
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Remove redundant CNAME creation (L113) as peaceiris/actions-gh-pages handles it with cname parameter - Simplify "Rebuild for smalruby3-gui GitHub Pages" step to only run postbuild.mjs instead of full npm run build - Simplify "Rebuild for branch GitHub Pages" step to only run postbuild.mjs instead of full npm run build - Remove unnecessary NODE_OPTIONS and NODE_ENV environment variables from rebuild steps These changes eliminate duplicate build processes and improve workflow efficiency since the initial webpack build already completed before these rebuild steps. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Replace touch build/.nojekyll with rm -f build/CNAME in smalruby3-gui GitHub Pages deployment step - .nojekyll already exists from smalruby.app deployment since they share the same build directory - Remove CNAME file created for smalruby.app to prevent domain conflicts with smalruby3-gui GitHub Pages 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
github-actions bot
pushed a commit
that referenced
this pull request
Sep 15, 2025
…-workflow ci: optimize GitHub Actions CI/CD workflow
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Implementation Details
Removed redundant CNAME creation (L113): The
echo "smalruby.app" > build/CNAMEcommand was redundant since peaceiris/actions-gh-pages already handles CNAME creation through thecnameparameterSimplified smalruby3-gui GitHub Pages rebuild (L123): Changed from
npm run buildtonpm exec node ./scripts/postbuild.mjssince the initial webpack build is already completeSimplified branch GitHub Pages rebuild (L143): Applied the same optimization as above
Removed unnecessary environment variables: Removed
NODE_OPTIONSandNODE_ENVfrom rebuild steps since they're only needed for the full build processTest Plan
🤖 Generated with Claude Code