coresoftware icon indicating copy to clipboard operation
coresoftware copied to clipboard

Fix g4tpc

Open pinkenburg opened this issue 7 months ago • 3 comments

Types of changes

  • [X ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work for users)
  • [ ] Requiring change in macros repository (Please provide links to the macros pull request in the last section)
  • [x] I am a member of GitHub organization of sPHENIX Collaboration, EIC, or ECCE (contact Chris Pinkenburg to join)

What kind of change does this PR introduce? (Bug fix, feature, ...)

This PR takes care of the various clang-tidy warnings in g4tpc

TODOs (if applicable)

Links to other PRs in macros and calibration repositories (if applicable)

pinkenburg avatar May 04 '25 17:05 pinkenburg

Build & test report

Report for commit 1df3a7ea57db7f39f14d7323e59da321a507dfb7: Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration sPHENIX             jenkins.io

sphenix-jenkins-ci[bot] avatar May 05 '25 01:05 sphenix-jenkins-ci[bot]

something went badly wrong, putting this PR on ice for the time being

pinkenburg avatar May 05 '25 13:05 pinkenburg

This can be the poster child showing clang-tidy actually does find problems in code...

osbornjd avatar May 05 '25 13:05 osbornjd