R3BRootGroup / R3BRoot

Framework for Simulations and Data Analysis of R3B Experiment
https://github.com/R3BRootGroup/R3BRoot/wiki
GNU General Public License v3.0
18 stars 104 forks source link

Clang-tidy config for neuland and rolu #906

Closed YanzhaoW closed 1 year ago

YanzhaoW commented 1 year ago

Info:

  1. Add separate clang-tidy configuration for NeuLAND and ROLU.
  2. The default .clang-tidy located in the project root folder is now a link to config/clang_tidy/global.yml with exactly the same configuration.
  3. Configurations for NeuLAND Related folders enable stricter rules (I will personal resolve those warnings for any NeuLAND related PRs).
  4. All clang-tidy warnings are disabled for ROLU related detector (as requested from #896) because currently no one is available to dedicate on software development of the detector.
  5. Add an instruction for the clang-tidy configuration.

Related to #907


Checklist:

YanzhaoW commented 1 year ago

Ok, all conflicts resolved. Change the old config to core.yml and apply it to the core utilities in R3BRoot. Put @klenze config to default.yml (which is basically everything disabled).

The core.yml is linked to .clang-tidy files in folder r3bbase and r3bsource/base.

The core configuration is, of course, subjected to any discussion, which should be focused on REASONS of programming and software (as the implementation of core functionalities already requires relative high level of programming).

jose-luis-rs commented 1 year ago

Thanks