Commits on Source (3)
-
Yen-Chen Chen authored
<!-- ICON --------------------------------------------------------------- Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss Contact information: icon-model.org See AUTHORS.TXT for a list of authors See LICENSES/ for license information SPDX-License-Identifier: CC0-1.0 --------------------------------------------------------------- --> ## What is the bug The header check macros are always true ## How do you fix it Make it configured to the same as the CMake variable, where the presence of the header is checked. ## How urgent is the bugfix - [ ] I need it as soon as possible - [x] I can wait for a couple of days - [ ] None of my current codes is directly affected ## Mandatory steps before review - [x] Gitlab CI passes _(Hint: use `make format` for linting)_ - [x] Bugfix is covered by additional unit tests - [x] Mark the merge request as ready by removing `Draft:` ## Mandatory steps before merge - [x] Test coverage does not decrease - [ ] Reviewed by a maintainer - [ ] Incorporate review suggestions - [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other) **You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!** Approved-by:
Jonas Jucker <jonas.jucker@env.ethz.ch> Merged-by:
Jonas Jucker <jonas.jucker@env.ethz.ch> Changelog: bugfix
-
Yen-Chen Chen authored
<!-- ICON --------------------------------------------------------------- Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss Contact information: icon-model.org See AUTHORS.TXT for a list of authors See LICENSES/ for license information SPDX-License-Identifier: CC0-1.0 --------------------------------------------------------------- --> ## What is the new feature Continue adding C tests. This MR closes #6 ## How is it implemented ## Mandatory steps before review - [x] Gitlab CI passes _(Hint: use `make format` for linting)_ - [x] New feature is covered by additional unit tests - [x] Mark the merge request as ready by removing `Draft:` ## Mandatory steps before merge - [x] Test coverage does not decrease - [x] Reviewed by a maintainer - [x] Incorporate review suggestions - [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other) **You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!** Approved-by:
Jonas Jucker <jonas.jucker@env.ethz.ch> Approved-by:
Terry Cojean <terry.cojean@kit.edu> Merged-by:
Yen-Chen Chen <yen-chen.chen@kit.edu> Changelog: other
-
Yen-Chen Chen authored
<!-- ICON --------------------------------------------------------------- Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss Contact information: icon-model.org See AUTHORS.TXT for a list of authors See LICENSES/ for license information SPDX-License-Identifier: CC0-1.0 --------------------------------------------------------------- --> ## What is the new feature _Please describe your feature in a couple of words._ ## How is it implemented _Describe important implementation details of the feature._ ## Mandatory steps before review - [x] Gitlab CI passes _(Hint: use `make format` for linting)_ - [x] New feature is covered by additional unit tests - [x] Mark the merge request as ready by removing `Draft:` ## Mandatory steps before merge - [x] Test coverage does not decrease - [x] Reviewed by a maintainer - [x] Incorporate review suggestions - [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other) **You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!** Approved-by:
Jonas Jucker <jonas.jucker@env.ethz.ch> Merged-by:
Jonas Jucker <jonas.jucker@env.ethz.ch> Changelog: other
Showing
- cmake/gtest_helper.cmake 2 additions, 2 deletionscmake/gtest_helper.cmake
- src/CMakeLists.txt 1 addition, 0 deletionssrc/CMakeLists.txt
- src/config.h.in 4 additions, 4 deletionssrc/config.h.in
- src/nml_annotate.h 25 additions, 0 deletionssrc/nml_annotate.h
- src/util_file.c 2 additions, 3 deletionssrc/util_file.c
- src/util_file.h 2 additions, 0 deletionssrc/util_file.h
- src/util_sysinfo.c 0 additions, 2 deletionssrc/util_sysinfo.c
- src/util_sysinfo.h 30 additions, 0 deletionssrc/util_sysinfo.h
- src/util_timer.h 31 additions, 0 deletionssrc/util_timer.h
- test/c/CMakeLists.txt 25 additions, 0 deletionstest/c/CMakeLists.txt
- test/c/ctest_nml_annotate.cpp 69 additions, 0 deletionstest/c/ctest_nml_annotate.cpp
- test/c/ctest_util_file.cpp 40 additions, 6 deletionstest/c/ctest_util_file.cpp
- test/c/ctest_util_hash.cpp 112 additions, 0 deletionstest/c/ctest_util_hash.cpp
- test/c/ctest_util_sysinfo.cpp 60 additions, 0 deletionstest/c/ctest_util_sysinfo.cpp
- test/c/ctest_util_timer.cpp 57 additions, 0 deletionstest/c/ctest_util_timer.cpp
- test/fortran/test_fortran_tools.f90 441 additions, 0 deletionstest/fortran/test_fortran_tools.f90
src/nml_annotate.h
0 → 100644
src/util_sysinfo.h
0 → 100644
src/util_timer.h
0 → 100644
test/c/ctest_nml_annotate.cpp
0 → 100644
test/c/ctest_util_sysinfo.cpp
0 → 100644
test/c/ctest_util_timer.cpp
0 → 100644