Page MenuHomeHEPForge

Get rid of EvtPatches.hh
ClosedPublic

Authored by abudinen on Jun 20 2024, 6:14 PM.
Tags
  • Restricted Project
Referenced Files
F19317052: D123.1759240215.diff
Mon, Sep 29, 2:50 PM
F19316647: D123.1759239394.diff
Mon, Sep 29, 2:36 PM
F19316315: D123.1759238644.diff
Mon, Sep 29, 2:24 PM
F19310856: D123.1759213294.diff
Mon, Sep 29, 7:21 AM
F19310482: D123.1759210936.diff
Mon, Sep 29, 6:42 AM
F19305971: D123.1759196389.diff
Mon, Sep 29, 2:39 AM
F19304781: D123.1759188387.diff
Mon, Sep 29, 12:26 AM
F19304267: D123.1759186340.diff
Sun, Sep 28, 11:52 PM
Subscribers
None

Details

Summary

This gets rid of the EvtPatches. Related with !24.

Test Plan

All tests are invariant under these changes. See results in

/home/epp/phsgfz/workingDirectory/RemoveEvtPatches/install/share/EvtGen/test/plots/*Pul*.

Diff Detail

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

Event Timeline

abudinen created this revision.
abudinen changed the visibility from "All Users" to "Public (No Login Required)".Jun 20 2024, 6:18 PM
abudinen changed the edit policy from "All Users" to "Restricted Project (Project)".
abudinen added a project: Restricted Project.
  • Removed trailing space.
  • Resolved conflicts in History.

Thanks @abudinen - just a minor wording suggestion for the History file.
Since these are my changes, I'll let John and/or Michal do the actual reviewing!

History.md
15
  • Applied wording comments for History.

Ok, waiting for @jback or @kreps approval.

This can be merged. Good to have this code cleanup.

This revision is now accepted and ready to land.Jun 21 2024, 12:33 PM

Closing by hand as it did not happen automatically after landing .