[BEAM-4555] Pre-commit file include triggering breaks phrase triggering

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: 2.6.0
• Component/s: testing
• Assignee: Scott Wegner
• Reporter: Scott Wegner
• Created: 13/Jun/18 20:31
• Updated: 16/May/20 13:47
• Resolved: 14/Jun/18 05:29

Description

In BEAM-4445 we changed pre-commit triggering on PR's to only run when afffected file paths were changed. However as a side-effect, it is no longer possible to use trigger phrases to manually run pre-commits if include path files are not changed.

This ability to run a pre-commit even if files are not changed is useful to check the stability of the codebase before submitting.

Issue Links

• is broken by: BEAM-4445 Filter pre-commit triggering based on touched files
• links to: GitHub Pull Request #5637
• links to: GitHub Pull Request #5638