Move flake8 checking to build stage

Remove the separate flake8 unit test of the python directory and instead use the dedicated feature of atlas_install_python_modules.

Some of these tests were only checking new-style configuration, but that's already the default, so no separate test is needed.

This merge request affects 16 packages:

• Event/DumpEventDataToJSON
• PhysicsAnalysis/DPDUtils
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkExotics
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkSUSY
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkTau
• PhysicsAnalysis/ElectronPhotonID/ElectronPhotonSelectorTools
• PhysicsAnalysis/NTUPtoNTUP/JetN2N
• PhysicsAnalysis/PyAnalysis/PyAnalysisUtils
• PhysicsAnalysis/TruthParticleID/McParticleTests
• Reconstruction/DiTauRec
• Reconstruction/RecExample/RecExAlgs
• Reconstruction/egamma/egammaConfig
• Reconstruction/egamma/egammaMVACalib
• Reconstruction/egamma/egammaTools
• Reconstruction/egamma/egammaTrackTools
• Tools/OnlineRecoTests

Adding @goetz ,@jdandoy ,@ykeisuke ,@crizzi ,@xiaozhon ,@alopezso ,@adbailey as watchers

added Analysis Derivation EDM Egamma JetEtmiss Reconstruction Tau Tools master review-pending-level-1 labels

CI Result SUCCESS (hash ecf0efde)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 13208]

The changes look ok. Adding review-approved label.

L1 shifter

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

merged

mentioned in commit 233af308

added sweep:ignore label