NCAR / micm

A model-independent chemistry module for atmosphere models
https://ncar.github.io/micm/
Apache License 2.0
6 stars 4 forks source link

Apply rules to order Include headers #476

Closed boulderdaze closed 4 months ago

boulderdaze commented 4 months ago

Closes #473

include style: llvm style guide

  1. main module header
  2. local/private headers
  3. project headers
  4. external libraries headers
  5. system headers

Main

#pragma once

#include <array>
#include <filesystem>
#include <fstream>
#include <iostream>
#include <micm/process/tunneling_rate_constant.hpp>
#include <micm/process/user_defined_rate_constant.hpp>
#include <micm/solver/rosenbrock.hpp>
#include <micm/system/system.hpp>
#include <micm/util/constants.hpp>
#include <micm/util/error.hpp>
#include <nlohmann/json.hpp>
#include <sstream>
#include <system_error>

473-order-headers

#pragma once

#include <micm/process/tunneling_rate_constant.hpp>
#include <micm/process/user_defined_rate_constant.hpp>
#include <micm/solver/rosenbrock.hpp>
#include <micm/system/system.hpp>
#include <micm/util/constants.hpp>
#include <micm/util/error.hpp>

#include <nlohmann/json.hpp>

#include <array>
#include <filesystem>
#include <fstream>
#include <iostream>
#include <sstream>
#include <system_error>
codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 75.00000% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 93.32%. Comparing base (d73f052) to head (69bb7ca). Report is 1 commits behind head on main.

Files Patch % Lines
examples/util_example.cpp 35.29% 11 Missing :warning:
examples/example.cpp 83.01% 9 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #476 +/- ## ========================================== - Coverage 93.35% 93.32% -0.04% ========================================== Files 40 40 Lines 3177 3177 ========================================== - Hits 2966 2965 -1 - Misses 211 212 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.