Skip to content

Please consider the following formatting changes to #12837#4

Closed
alibuild wants to merge 2 commits intocima22:oldCPUTPCDecodingSettingfrom
alibuild:alibot-cleanup-12837
Closed

Please consider the following formatting changes to #12837#4
alibuild wants to merge 2 commits intocima22:oldCPUTPCDecodingSettingfrom
alibuild:alibot-cleanup-12837

Conversation

@alibuild
Copy link

@alibuild alibuild commented Mar 8, 2024

This PR cannot be merged as is. You should either run clang-format yourself and update the pull request, or merge this PR in yours.
You can find the AliceO2 coding conventions at https://github.com/AliceO2Group/CodingGuidelines.

@cima22 cima22 force-pushed the oldCPUTPCDecodingSetting branch from e12b28e to b5a5d96 Compare March 8, 2024 14:02
@alibuild alibuild closed this Mar 8, 2024
@alibuild alibuild deleted the alibot-cleanup-12837 branch March 8, 2024 14:03
cima22 pushed a commit that referenced this pull request Mar 12, 2024
* MCH standalone alignment module

with a common part(for both MCH and MTF) of mathematics basic codes moved to a common path under O2/Detectors/ForwardAlign

Co-Authored-By: javierecc <javier.castillo.castellanos@cern.ch>

* Please consider the following formatting changes (#4)

* Removed from PR

* Fixing formatting issue

* Add missing header for filesystem

* Fix warning with dereferenced iterator

* Remove re-initialisation of magfield for trackfitter and move residual QA plots to O2Physics task

* Update comment for track selection

* Remove unused function

* Remove re-definition of magfield which is set already with CCDB finalisation

* Fixing CI build errors

---------

Co-authored-by: javierecc <javier.castillo.castellanos@cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants