ISF_FastCaloSim : clang-tidy

ISF_FastCaloSim : clang-tidy

This merge request affects 4 packages:

• Simulation/ISF/ISF_FastCaloSim/ISF_FastCaloSimEvent
• Simulation/ISF/ISF_FastCaloSim/ISF_FastCaloSimSD
• Simulation/ISF/ISF_FastCaloSim/ISF_FastCaloSimServices
• Simulation/ISF/ISF_FastCaloSim/ISF_PunchThroughTools

Affected files list will not be printed in this case

Adding @jchapman ,@pjacka ,@mfauccig ,@xiaozhon ,@ahasib ,@tadej ,@schaarsc as watchers

added Simulation master review-pending-level-1 labels

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

172K in file Simulation/ISF/ISF_FastCaloSim/ISF_FastCaloSimEvent/src/TFCS1DFunctionRegression.cxx

CI Result SUCCESS (hash 564caff2)

	Athena
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 67427]

added review-user-action-required label and removed review-pending-level-1 label

removed review-user-action-required label

added review-pending-level-1 label

My comments need not block the MR so putting it back on review and resolving threads.

resolved all threads

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

added review-approved-point1 label

merged

mentioned in commit 82ac2f2b

added sweep:ignore label