Skip to content

Conversation

@vim-zz
Copy link
Collaborator

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

This adds a new warning box explaining when implicit triggers are disabled, clarifies that include/exclude filters don't change triggering events, and improves several explanations throughout the execution model documentation to reduce confusion about trigger behavior.

Screenshot 2025-11-12 at 16 12 10 Screenshot 2025-11-12 at 16 12 24

✨ PR Description

Purpose: Improve trigger documentation by clarifying the difference between explicit and implicit triggering mechanisms in gitStream platform.

Main changes:

  • Added explanation of how implicit triggers remain active when using only include/exclude parameters
  • Clarified that explicit triggers (on parameter) completely override default implicit trigger behavior
  • Enhanced documentation with explanatory notes about trigger behavior across different platforms
  • Reformatted warning sections for better readability and added context notes to code examples

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

This adds a new warning box explaining when implicit triggers are
disabled, clarifies that `include`/`exclude` filters don't change
triggering events, and improves several explanations throughout the
execution model documentation to reduce confusion about trigger
behavior.
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

LGTM

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 review from a team November 12, 2025 14:17
gitstream-cm[bot]
gitstream-cm bot previously approved these changes Nov 12, 2025
@gitstream-cm gitstream-cm bot dismissed their stale review November 12, 2025 14:17

Review dismissed

@vim-zz vim-zz merged commit 3af35c1 into main Nov 12, 2025
11 checks passed
@vim-zz vim-zz deleted the explicit-triggers-and-default branch November 12, 2025 14:17
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