Skip to content
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.

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

48 refactor with clang tidy or sonarlint #50

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

aquaherd
Copy link
Member

Nothing severe has been found but lots of minor findings to improve code quality and ease of use.

aquaherd and others added 6 commits March 28, 2023 20:26
- more sonarlint findings
- made debug output silent but possible
- more automake leftovers removed
- found debug warnings from gtk in view and settings
- triggered po rebuild
- add deb packaging
- add rpm packaging
- update README.md
Label callback did no longer handle 'inactive' triggered by
Facts-Changed dBus event.
- add .editorconfig and .clangd files
- update copyright notes
@aquaherd aquaherd linked an issue Nov 19, 2023 that may be closed by this pull request
@aquaherd aquaherd marked this pull request as draft November 19, 2023 13:04
@aquaherd aquaherd force-pushed the 48-refactor-with-clang-tidy-or-sonarlint branch from cc6bbe5 to e370cf9 Compare November 19, 2023 13:05
@aquaherd aquaherd force-pushed the 48-refactor-with-clang-tidy-or-sonarlint branch from e370cf9 to 63e81fa Compare November 19, 2023 13:50
@aquaherd aquaherd marked this pull request as ready for review March 2, 2024 17:56
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.

Refactor with clang-tidy or sonarlint
1 participant