-
Notifications
You must be signed in to change notification settings - Fork 658
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
refactor(traffic_light_utils): prefix package and namespace with autoware #9251
refactor(traffic_light_utils): prefix package and namespace with autoware #9251
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
fd99f17
to
4718445
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9251 +/- ##
==========================================
- Coverage 29.78% 29.78% -0.01%
==========================================
Files 1338 1343 +5
Lines 103302 103369 +67
Branches 40240 40253 +13
==========================================
+ Hits 30770 30786 +16
- Misses 69547 69597 +50
- Partials 2985 2986 +1
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
015669b
to
f9220b1
Compare
@ktro2828 @maxime-clem thanks @mitsudome-r @xmfcx could you merge this PR? @ktro2828 is a maintainer for this packages and he has approved the changes. Thanks. |
The CODEOWNERS is conflicting, could you fix it? By the way, in these PRs, you don't need to manually update the CODEOWNERS files. Instead, you can first merge the PR and then run https://github.com/autowarefoundation/autoware.universe/actions/workflows/update-codeowners-from-packages.yaml on the main and it will automatically create a PR to update them. And you can review that PR and merge it. (don't forget to add Here is an example PR autogenerated by it: You can find them under this tag: |
…ware Signed-off-by: Esteve Fernandez <[email protected]>
f9220b1
to
e3216dc
Compare
@xmfcx I've fixed the conflicts, thanks for the heads up about the autogenerated |
Description
prefix package and namespace with
autoware
Related links
Parent Issue:
autoware.universe/common
packages autoware#5077How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.