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

feat!: split package into description and launch #23

Closed
wants to merge 6 commits into from

Conversation

kenji-miyake
Copy link
Contributor

@kenji-miyake kenji-miyake commented Feb 19, 2022

Since this breaks the package paths written in autoware_launch, we need to merge the PR to autoware_launch at the same time.

Related: tier4/autoware_launch#213

@kenji-miyake
Copy link
Contributor Author

Also, we have to change other vehicles as well.

@wep21
Copy link
Contributor

wep21 commented Feb 19, 2022

@kenji-miyake "!" in the title is typo?

@kenji-miyake
Copy link
Contributor Author

@wep21 No, it tells people there is a breaking change.
https://www.conventionalcommits.org/en/v1.0.0/

@wep21
Copy link
Contributor

wep21 commented Feb 19, 2022

@kenji-miyake I got it. Thanks!

@yukkysaito
Copy link
Collaborator

LGTM 👍

Kenji Miyake and others added 5 commits February 21, 2022 14:30
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
fix
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Co-authored-by: Daisuke Nishimatsu <42202095+wep21@users.noreply.github.com>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
@kenji-miyake
Copy link
Contributor Author

kenji-miyake commented Feb 21, 2022

Rebased to resolve conflicts.

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
@h-ohta
Copy link
Contributor

h-ohta commented Nov 8, 2022

I'll close and make another PR to do same thing for latest.

@h-ohta h-ohta closed this Nov 8, 2022
@kenji-miyake kenji-miyake deleted the split-package branch November 8, 2022 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants