Skip to content

Fixes - #4

Closed
AndreaRicchi wants to merge 6 commits into
developfrom
fixes
Closed

Fixes#4
AndreaRicchi wants to merge 6 commits into
developfrom
fixes

Conversation

@AndreaRicchi

Copy link
Copy Markdown
Contributor

No description provided.

@github-actionsgithub-actionsBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️Clang-Tidy found issue(s) with the introduced code (1/1)

Comment threadsrc/dbus/gdbus_private.hpp Outdated
@AndreaRicchi
AndreaRicchiforce-pushed the fixes branch 7 times, most recently from aa38791 to a2461b6CompareSeptember 10, 2025 14:58

@github-actionsgithub-actionsBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️Clang-Tidy found issue(s) with the introduced code (1/1)

Comment threadsrc/dbus/gdbus_private.hpp Outdated
@AndreaRicchi
AndreaRicchiforce-pushed the fixes branch 5 times, most recently from f72c686 to 965341dCompareSeptember 11, 2025 08:25
Signed-off-by: Andrea Ricchi <andrea.ricchi@amarulasolutions.com>
Added GitHub action to build and check cpp code.
Signed-off-by: Andrea Ricchi <andrea.ricchi@amarulasolutions.com>
Added pre-commit configuration for code style and formatting.
Signed-off-by: Andrea Ricchi <andrea.ricchi@amarulasolutions.com>
Signed-off-by: Andrea Ricchi <andrea.ricchi@amarulasolutions.com>
Signed-off-by: Andrea Ricchi <andrea.ricchi@amarulasolutions.com>

@EddyTheCoEddyTheCo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This action will not work from forks or at least it is not working now.
This is not needed the repo already have clang-tidy review comments actions.

Sign up for freeto 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

@AndreaRicchi@EddyTheCo