Skip to content

Remove deprecated fmt::localtime API and update fmt to 12.0.0 #1973

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

Open
njfox wants to merge 4 commits into
base: main
Choose a base branch
from

update to fmt 12.0.0

7b90c96
Select commit
Loading
Failed to load commit list.
Open

Remove deprecated fmt::localtime API and update fmt to 12.0.0 #1973

update to fmt 12.0.0
7b90c96
Select commit
Loading
Failed to load commit list.
CodeScene Delta Analysis / CodeScene Code Health Review (main) succeeded Oct 24, 2025 in 36s

CodeScene PR Check

Quality Gate Passed

Gates Passed
4 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

Details

✅ Improving Code Health:

  • Lines of Code in a Single File gui.cc 🔥

Annotations

Check notice on line 1 in src/gui/gui.cc

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Code Health Review (main)

✅ Getting better: Lines of Code in a Single File 

The lines of code decreases from 2464 to 2463, improve code health by reducing it to 1000. The number of Lines of Code in a single file. More Lines of Code lowers the code health.
View more details on CodeScene Delta Analysis