Skip to content

Comments

fix: update Claude Code Review workflow for improved functionality#69

Merged
Gkrumbach07 merged 4 commits intoambient-code:mainfrom
Gkrumbach07:bug/fix-claude-pr-review
Sep 11, 2025
Merged

fix: update Claude Code Review workflow for improved functionality#69
Gkrumbach07 merged 4 commits intoambient-code:mainfrom
Gkrumbach07:bug/fix-claude-pr-review

Conversation

@Gkrumbach07
Copy link
Collaborator

  • Adjusted checkout step to use pull request merge reference and disabled credential persistence

- Adjusted checkout step to use pull request merge reference and disabled credential persistence.
- Cleaned up whitespace for better readability.
- Enhanced guidance on using the repository's CLAUDE.md for style and conventions.

These changes aim to streamline the code review process and ensure proper integration with GitHub actions.
- Changed the workflow trigger from 'pull_request' to 'pull_request_target' to enhance security and functionality.
- Changed pull-requests permission from 'read' to 'write' to allow for enhanced interaction with pull requests during the review process.
@cooktheryan
Copy link
Collaborator

/lgtm
/approve

@Gkrumbach07 Gkrumbach07 merged commit f026101 into ambient-code:main Sep 11, 2025
1 check passed
sallyom pushed a commit that referenced this pull request Oct 15, 2025
fix: update Claude Code Review workflow for improved functionality
@bobbravo2 bobbravo2 added this to the v0.0.1 milestone Jan 30, 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.

3 participants