CPAlgorithms: Fix trigger matching selection name
Bringing back the naming convention that was lost in !69978 (merged)
It needs to match what is set here in order to be able to use the GLOBALTRIGMATCH
keyword in the event selection.
Merge request reports
Activity
Hi @ravinab,
I think in general this seems fine. I'm not quite sure what that decoration does, but obviously decorations should be more explicit in their name than
dontsave
. Approving for AR. If you think users may rely on that decoration (given the odd name I wouldn't think so), please add the analysis-interface-change label. If this is fixing a bug, please add the bugfix label.If you feel this is decoration is widely used (seems unlikely), we would have to discuss the merit of the change.
Cheers, Nils [Analysis RC]
added analysis-review-approved label and removed analysis-review-required label
added bugfix label
CI Result SUCCESS (hash 0988c81b)Athena AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9265] (remote access info)added review-approved label and removed review-pending-level-1 label
mentioned in commit c699bb99