Skip to content

fix: use commit-message input in create-pr action#56

Merged
carlosthe19916 merged 1 commit into
guacsec:mainfrom
carlosthe19916:fix/create-pr-commit-message
Jul 8, 2026
Merged

fix: use commit-message input in create-pr action#56
carlosthe19916 merged 1 commit into
guacsec:mainfrom
carlosthe19916:fix/create-pr-commit-message

Conversation

@carlosthe19916

Copy link
Copy Markdown
Contributor

Summary

  • The commit-message parameter in the create-pr composite action was hardcoded to "update client/openapi/trustd.yaml" instead of using ${{ inputs.commit-message }}
  • This caused commitlint to fail with subject-empty and type-empty errors because the hardcoded string didn't follow conventional commit format
  • Now correctly passes through the caller-provided commit message

Test plan

  • Trigger a workflow that uses the create-pr action and verify the commit message matches the input
  • Verify commitlint passes on the resulting PR

🤖 Generated with Claude Code

… action

The commit-message was hardcoded to "update client/openapi/trustd.yaml"
instead of using the inputs.commit-message parameter, causing commitlint
to fail with subject-empty and type-empty errors.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@carlosthe19916 carlosthe19916 merged commit 835f65f into guacsec:main Jul 8, 2026
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.

1 participant