Lift trigger_request association up to Ci::Processable
What does this MR do and why?
Both Ci::Build
and Ci::Bridge
belong to trigger_request
and share
variable builder methods that depend on it. We recently added
CI_TRIGGER_SHORT_TOKEN
, but the necessary method delegation was only
added on Ci::Build
, which caused a problem when when a Ci::Bridge
was triggered.
The immediate fix was to add the missing delegation, but in a follow-up
discussion we decided that the association and similar delegations
should probably be on Ci::Processable
instead.
See #457571 (closed)
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.