Page MenuHomeHEPForge

Remove "reference" field from JSON test files
ClosedPublic

Authored by abudinen on Jun 18 2024, 10:20 PM.

Details

Summary

Instead simply prepend "Ref/" to output ROOT file name. Related with !23.

Cherry-picked 0adb52b6c0abe3bcdcf57db8698f09f72a84dae0 as it can be singled out nicely from !22 to make that MR lighter.

This is related with the general thread-safety modifications: T168

Test Plan

Output of tests is exactly the same, only reference location changed.

Diff Detail

Repository
rEVTGEN evtgen
Branch
RemoveRefFromJson
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 314
Build 314: arc lint + arc unit

Event Timeline

abudinen created this revision.
abudinen changed the visibility from "All Users" to "Public (No Login Required)".
abudinen changed the edit policy from "All Users" to "Restricted Project (Project)".
abudinen added a project: Restricted Project.

Looks good to me but then they are my changes ;)
So probably best to have them checked by @jback and/or @kreps before merging.

This revision is now accepted and ready to land.Jun 19 2024, 11:30 AM