Skip to content

Conversation

@takaokouji
Copy link

Summary

  • Fix BRANCH_NAME setting to use actual PR branch name instead of 'merge'
  • Add conditional execution for lint/test steps to optimize CI performance on main branches
  • Improve browser check step with proper conditional logic

Changes

  • Updated line 127 to use ${{ github.head_ref || github.ref_name }} for proper branch name detection
  • Added conditional if statements to skip lint and test steps on develop/main/master branches
  • Improved browser check step organization

Test plan

  • Verify PR deployments use correct branch names in GitHub Pages paths
  • Confirm main branch deployments still work correctly
  • Test that conditional steps execute properly on feature branches

🤖 Generated with Claude Code

- Fix BRANCH_NAME to use actual PR branch name instead of 'merge'
- Add conditional execution for lint and test steps to skip on main branches
- Improve browser check step with proper conditional logic

This ensures that PR deployments use the correct branch name for
GitHub Pages deployment paths.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@takaokouji takaokouji merged commit 15fd081 into develop Aug 24, 2025
2 checks passed
@takaokouji takaokouji deleted the fix/ci-cd-branch-name-improvement branch August 24, 2025 22:52
github-actions bot pushed a commit that referenced this pull request Aug 24, 2025
…nch-name-improvement

fix: improve CI/CD workflow branch name handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants