Skip to content

Improve Settings logLevelToInt - #9283

Merged
nbolton merged 5 commits into
masterfrom
settingsCleanup_2
Dec 18, 2025
Merged

Improve Settings logLevelToInt#9283
nbolton merged 5 commits into
masterfrom
settingsCleanup_2

Conversation

@sithlord48

@sithlord48 sithlord48 commented Dec 17, 2025

Copy link
Copy Markdown
Member
  • Settings::logLevelToInt simplify the method
  • Add tests for Settings logLevelToInt
  • typo fix peroid -> period
  • typo fix replace settings with state in setState logging
  • remove unneeded { } in settings::defaultValues
  • Add Q_OBJECT to validators with translations

@sithlord48
sithlord48 requested a review from a team December 17, 2025 03:40
@sithlord48
sithlord48 requested a review from nbolton as a code owner December 17, 2025 03:40
@nbolton
nbolton merged commit 2a54f57 into master Dec 18, 2025
34 checks passed
@nbolton
nbolton deleted the settingsCleanup_2 branch December 18, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants