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.

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

Jump to bottom

Add more checks from clang-tidy #3959

Merged
merged 7 commits into from
Jul 17, 2023
Merged

Add more checks from clang-tidy #3959

merged 7 commits into from
Jul 17, 2023

Conversation

sylvestre
Copy link
Contributor

No description provided.

@sylvestre
Copy link
Contributor Author

I don't think this error is mine but happy to fix it here if needed

Checkers REMOVED from upstream, with CodeChecker still assigning labels:

@bruntib
Copy link
Contributor

bruntib commented Jul 17, 2023

The error is not because of the first message, but because of the second:

Checkers ADDED to upstream, without labels in CodeChecker:

Thanks for the maintenance!

@bruntib bruntib merged commit 7428690 into Ericsson:master Jul 17, 2023
8 of 9 checks passed
@sylvestre sylvestre deleted the more-check branch July 17, 2023 11:12
@sylvestre
Copy link
Contributor Author

happy to help :)

@bruntib bruntib added this to the release 6.23.0 milestone Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bruntib bruntib bruntib approved these changes

@vodorok vodorok Awaiting requested review from vodorok vodorok is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
release 6.23.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sylvestre @bruntib