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.

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(operation_mode_transition_manager): prefix package and namespace with autoware_ #7291

Conversation

zulfaqar-azmi-t4
Copy link
Contributor

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 commented Jun 6, 2024

Description

This PR adds the autoware_ prefix to the package and puts headers in the autoware namespace.

Part of:

Tests performed

Build success and autoware is running.

Effects on system behavior

Not applicable.

Interface changes

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@github-actions github-actions bot added type:documentation Creating or refining documentation. (auto-assigned) component:control Vehicle control algorithms and mechanisms. (auto-assigned) component:launch Launch files, scripts and initialization tools. (auto-assigned) labels Jun 6, 2024
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 added the tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) label Jun 6, 2024
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 force-pushed the RT1-6682-autoware-prefix-for-operation-mode-transition-manager branch from 6e51340 to dc97a21 Compare June 6, 2024 05:54
@@ -1,5 +1,5 @@
<package format="3">
<name>operation_mode_transition_manager</name>
<name>autoware_operation_mode_transition_manager</name>
<version>0.1.0</version>
<description>The vehicle_cmd_gate package</description>
Copy link
Contributor

Choose a reason for hiding this comment

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

Not really related to this PR, but think the description is wrong.
Just to let you know. (I realized since I was modifying the vehicle_cmd_gate)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't notice it!
Thank you Sakayori-san. I'll just change them in this PR.

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 force-pushed the RT1-6682-autoware-prefix-for-operation-mode-transition-manager branch from 3b68025 to 31b0c14 Compare June 10, 2024 07:39
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 enabled auto-merge (squash) June 10, 2024 07:39
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 force-pushed the RT1-6682-autoware-prefix-for-operation-mode-transition-manager branch from 31b0c14 to 513c339 Compare June 11, 2024 05:03
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 added the tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Jun 11, 2024
Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
Signed-off-by: Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 force-pushed the RT1-6682-autoware-prefix-for-operation-mode-transition-manager branch from 513c339 to b8a2bdc Compare June 11, 2024 06:17
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 merged commit dcf6b81 into autowarefoundation:main Jun 11, 2024
23 of 24 checks passed
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 deleted the RT1-6682-autoware-prefix-for-operation-mode-transition-manager branch June 11, 2024 06:20
KhalilSelyan pushed a commit that referenced this pull request Jul 22, 2024
…pace with autoware_ (#7291)

* RT1-6682 add prefix package and namespace with autoware_

Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>

* RT1-6682 fix package's description

Signed-off-by: Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>

---------

Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
Signed-off-by: Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:control Vehicle control algorithms and mechanisms. (auto-assigned) component:launch Launch files, scripts and initialization tools. (auto-assigned) tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) type:documentation Creating or refining documentation. (auto-assigned)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants