-
Notifications
You must be signed in to change notification settings - Fork 557
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
fix(autoware_auto_common): move headers to a separate directory #5919
Conversation
9d98964
to
fa7e770
Compare
Hi @esteve Could you fix required CI fail...? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5919 +/- ##
=======================================
Coverage 15.23% 15.23%
=======================================
Files 1751 1755 +4
Lines 120918 120929 +11
Branches 36770 36773 +3
=======================================
+ Hits 18419 18429 +10
+ Misses 81825 81824 -1
- Partials 20674 20676 +2
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@satoshi-ota my apologies, I've fixed the errors in the latest commit. This PR is now ready for review. |
Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp>
ac41eae
to
8abb46a
Compare
@satoshi-ota I've rebased this PR on top of main and fixed the conflicts, could you have a look at it again? Thanks. |
…warefoundation#5919) * fix(autoware_auto_common): move headers to a separate directory Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…warefoundation#5919) * fix(autoware_auto_common): move headers to a separate directory Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…warefoundation#5919) * fix(autoware_auto_common): move headers to a separate directory Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…warefoundation#5919) * fix(autoware_auto_common): move headers to a separate directory Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Description
This PR moves headers to a seaparate
autoware_auto_common
directory.See autowarefoundation/autoware#4070
Tests performed
Not applicable.
Effects on system behavior
Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.