Skip to content

Sonar cleanup 17 - #8944

Merged
nbolton merged 2 commits into
masterfrom
sonarCleanup_17
Sep 11, 2025
Merged

Sonar cleanup 17#8944
nbolton merged 2 commits into
masterfrom
sonarCleanup_17

Conversation

@sithlord48

@sithlord48 sithlord48 commented Sep 11, 2025

Copy link
Copy Markdown
Member

Two small cleanups

  • Remove Arch::hostname and just call QSysInfo::machineHostName in the one place its used (should have done this to being with)
  • mark LogDock::eventFilter an override.

@sithlord48
sithlord48 requested a review from nbolton September 11, 2025 13:35
@nbolton
nbolton enabled auto-merge (rebase) September 11, 2025 13:45
@nbolton
nbolton merged commit 2f2754a into master Sep 11, 2025
31 checks passed
@nbolton
nbolton deleted the sonarCleanup_17 branch September 11, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nbolton nbolton nbolton approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sithlord48 @nbolton