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

Move logging backend to spdlog #23

Draft

hikinggrass wants to merge 47 commits into main from feature/spdlog

Draft

Move logging backend to spdlog #23

Move coverage targt setup to tests subdir

Codacy Production / Codacy Static Code Analysis required action Sep 3, 2024 in 0s

4 new issues (0 max.) of at least minor severity.

Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 4
           

Clones added
============
- tests/liblog_test.cpp  10
- lib/logging.cpp  4

See the complete overview on Codacy

Annotations

Check warning on line 11 in .ci/build-kit/scripts/run_unit_tests.sh

codacy-production / Codacy Static Code Analysis

.ci/build-kit/scripts/run_unit_tests.sh#L11

Double quote to prevent globbing and word splitting.

Check notice on line 74 in include/everest/logging.hpp

codacy-production / Codacy Static Code Analysis

include/everest/logging.hpp#L74

Condition 'log_file' is always false

Check notice on line 77 in include/everest/logging.hpp

codacy-production / Codacy Static Code Analysis

include/everest/logging.hpp#L77

Condition '!log_file' is always true

Check notice on line 89 in lib/logging.cpp

codacy-production / Codacy Static Code Analysis

lib/logging.cpp#L89

The scope of the variable 'cmdline' can be reduced.

View more details on Codacy Production