New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove references to nlohmann json in docker actions #675

Open

K20shores opened this issue Oct 8, 2024 · 0 comments

Labels

Milestone

MICM, TUV-x in CA...

Collaborator

K20shores commented Oct 8, 2024

Our clang-tidy species some paths to the old nlohmann::json library that we no longer use. remove them

micm/.github/workflows/clang-tidy.yml

Line 29 in fc0f8c4

    
                     docker run --name include-scans -e INCLUDE_FILES="${INCLUDE_FILES}" -t micm-clang-tidy bash -c 'time clang-tidy -p ./build/ --config-file="./.clang-tidy" -header-filter=$(pwd)/include/.* --extra-arg "-lcudart_static" --extra-arg "-std=c++20" ${INCLUDE_FILES} -- -Iinclude/ -isystem ./cuda-build/_deps/json-src/include'

K20shores added this to the MICM, TUV-x in CAM-SIMA milestone

K20shores added bug enhancement and removed bug labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment