Skip to content

Conversation

@strickvl
Copy link
Contributor

@strickvl strickvl commented Dec 12, 2025

Summary

  • Merges claude.yml and claude-code-review.yml into a single workflow
  • Simplifies trigger conditions by removing the /full-review exclusion
  • Uses an adaptive prompt that handles both general questions and code reviews

Changes

  • Removed .github/workflows/claude-code-review.yml (no longer needed)
  • Updated .github/workflows/claude.yml:
    • Removed !contains(..., '@claude /full-review') from all trigger conditions
    • Added adaptive prompt that helps with any request
    • Applies detailed review guidelines when user asks for a review

How it works

The prompt now tells Claude to:

  1. Help the user with whatever they asked
  2. If it looks like a review request (@claude /full-review, @claude review this, etc.), use the detailed review guidelines
  3. Otherwise, just be a helpful assistant

(FYI @htahir1 this removes the requirement for precisely following the exact syntax with / etc. I saw it not being used correctly a lot this past week. I'll replicate this over onto the frontend repos as well.)

Combines claude.yml and claude-code-review.yml into one workflow that:
- Triggers on any @claude mention (removes /full-review exclusion)
- Uses an adaptive prompt that helps with general questions
- Applies detailed review guidelines when review is requested
- Keeps all gating (team member check, bot exclusion)
- Preserves permissions and allowed tools
@strickvl strickvl added enhancement New feature or request internal To filter out internal PRs and issues x-squad Issues that are being handled by the x-squad no-release-notes Release notes will NOT be attached and used publicly for this PR. labels Dec 12, 2025
@strickvl strickvl requested a review from safoinme December 12, 2025 12:45
@strickvl strickvl moved this to In Review in ZenML Roadmap Dec 12, 2025
@strickvl strickvl self-assigned this Dec 12, 2025
@github-actions
Copy link
Contributor

Documentation Link Check Results

Absolute links check failed
There are broken absolute links in the documentation. See workflow logs for details
Relative links check passed
Last checked: 2025-12-12 13:17:07 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request internal To filter out internal PRs and issues no-release-notes Release notes will NOT be attached and used publicly for this PR. x-squad Issues that are being handled by the x-squad

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants