do not refer to classes which were removed from the release

related to ATEAM-725 although it is not fixing the warnings, just cleanup

WARNING: big files (>100K) are found in the changeset

140K in file Trigger/TriggerCommon/TrigEDMConfig/python/TriggerEDMRun2.py

This merge request affects 1 package:

• Trigger/TriggerCommon/TrigEDMConfig

This merge request affects 1 file:

• Trigger/TriggerCommon/TrigEDMConfig/python/TriggerEDMRun2.py

Adding @sutt ,@okuprash ,@bernius as watchers

added Trigger TriggerEDM master review-pending-level-1 labels

added 1 commit

• 6cd1c1e1 - remove also references from TrigDecisionTool and TriggerObjectMatching

Compare with previous version

WARNING: big files (>100K) are found in the changeset

140K in file Trigger/TriggerCommon/TrigEDMConfig/python/TriggerEDMRun2.py

This merge request affects 3 packages:

• Trigger/TrigAnalysis/TrigDecisionTool
• Trigger/TrigAnalysis/TrigObjectMatching
• Trigger/TriggerCommon/TrigEDMConfig

Affected files list will not be printed in this case

Adding @tamartin ,@sutt ,@okuprash ,@bernius as watchers

CI Result SUCCESS (hash 6cd1c1e1)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30898]

Hi @jmasik, is the plan to remove the references at some point or do you need to leave them commented?

Thanks, Marianna

Hi @liberato I'd prefer to keep them commented in the code, it is a special case, we normally don't remove any classes which were part of the edm cheers Jiri

Hi @jmasik, thanks for the clarification! Then I will approve the MR. Cheers, Marianna

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit a8c03108

added sweep:ignore label