Skip to content

Conversation

@vim-zz
Copy link
Collaborator

@vim-zz vim-zz commented Nov 3, 2025

Standardize condition descriptions to consistently reference context usage rather than specific properties. Move GitHub icons to the end of trigger names for better readability.

Screenshot 2025-11-03 at 15 44 24 Screenshot 2025-11-03 at 15 46 29

✨ PR Description

Purpose: Update PR trigger condition descriptions to use consistent "pr context" terminology and improve clarity of event handling rules.

Main changes:

  • Standardized condition descriptions to use "when pr context is used in .cm" format
  • Updated approval trigger conditions to include specific action dependencies
  • Fixed text formatting for GitHub-specific events for better visual consistency

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

Standardize condition descriptions to consistently reference context
usage rather than specific properties. Move GitHub icons to the end of
trigger names for better readability.
Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

Copy link
Contributor

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✨ PR Review

The PR standardizes trigger condition descriptions and improves formatting, but introduces a significant loss of precision in the technical documentation that could mislead users.

1 issues detected:

🧾 Readability - Generic condition descriptions lose important specificity about trigger activation requirements.

Details: The standardization from specific property references (pr.comment, pr.description, pr.label, pr.title) to generic "pr context is used in .cm" removes critical technical details about exactly which context properties trigger these events. This could mislead users about when their automations will actually execute.
File: docs/execution-model.md

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how

@gitstream-cm gitstream-cm bot requested a review from a team November 3, 2025 13:48
gitstream-cm[bot]
gitstream-cm bot previously approved these changes Nov 3, 2025
@gitstream-cm gitstream-cm bot dismissed their stale review November 3, 2025 13:49

Review dismissed

@vim-zz vim-zz merged commit 120e0e7 into main Nov 3, 2025
11 checks passed
@vim-zz vim-zz deleted the triggers-correct-context branch November 3, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant