Skip to content

Conversation

al1img
Copy link
Collaborator

@al1img al1img commented Oct 10, 2025

No description provided.

Copy link
Member

@mlohvynenko mlohvynenko left a comment

Choose a reason for hiding this comment

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

Reviewed-by: Mykhailo Lohvynenko <[email protected]>

#include <core/common/types/common.hpp>

namespace aos::iamclient {
/**
Copy link
Member

Choose a reason for hiding this comment

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

Add an empty line

Copy link

codecov bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.65%. Comparing base (b44aabc) to head (b1cad61).

Additional details and impacted files
@@                 Coverage Diff                  @@
##           feature_unification     #385   +/-   ##
====================================================
  Coverage                81.65%   81.65%           
====================================================
  Files                      201      201           
  Lines                    21959    21959           
  Branches                  3053     3053           
====================================================
  Hits                     17931    17931           
  Misses                    4028     4028           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@mykola-kobets-epam mykola-kobets-epam left a comment

Choose a reason for hiding this comment

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

Reviewed-by: Mykola Kobets <[email protected]>

Signed-off-by: Oleksandr Grytsov <[email protected]>
Reviewed-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Most of headers doesn't have this comment and it is hard to update when name
or path of header changed. Remove it to align with other headers.

Signed-off-by: Oleksandr Grytsov <[email protected]>
Reviewed-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants