Skip to content

Fix call to checkWhiteList

Spyros Argyropoulos requested to merge fix2 into master

Description of bug

checkWhiteList was called without the args.DRY_RUN argument which created an issue because of the following lines in jo_utils.py

DIR = r'.*' if dryRun else ALLOWED_PATHS
isWhitelisted = any([ re.match('{}/{}'.format(DIR, tag), path) for tag in re_checks ])

Changes introduced

Add the argument above

Tests

Before

	File: ../newGen21/newForcedFxFx/tmp501716/mc.MGPy8EG_Wenu_FxFx_3jets_HT2bias_ForceDplusD0.py cannot be added to the commit. Skipping.
([0-9]{3}xxx/[0-9]{6}|[0-9]{1}/[0-9]{4}xxx/[0-9]{7}) , ../newGen21/newForcedFxFx/tmp501716/merge_LHE.py
		File: ../newGen21/newForcedFxFx/tmp501716/merge_LHE.py cannot be added to the commit. Skipping.
([0-9]{3}xxx/[0-9]{6}|[0-9]{1}/[0-9]{4}xxx/[0-9]{7}) , ../newGen21/newForcedFxFx/tmp501716/DplusAndMinusToKpipiAndD0ToKpi.dec
		File: ../newGen21/newForcedFxFx/tmp501716/DplusAndMinusToKpipiAndD0ToKpi.dec cannot be added to the commit. Skipping.

After

Will add: ../newGen21/newForcedFxFx/tmp501716/mc.MGPy8EG_Wenu_FxFx_3jets_HT2bias_ForceDplusD0.py
.* , ../newGen21/newForcedFxFx/tmp501716/merge_LHE.py
		Will add: ../newGen21/newForcedFxFx/tmp501716/merge_LHE.py
.* , ../newGen21/newForcedFxFx/tmp501716/DplusAndMinusToKpipiAndD0ToKpi.dec
		Will add: ../newGen21/newForcedFxFx/tmp501716/DplusAndMinusToKpipiAndD0ToKpi.dec

Maybe @mshapiro you can also test this?

Issues resolved

Closes #

Merge request reports