Skip to content

Add support for Kilted#804

Merged
christophebedard merged 3 commits intomasterfrom
christophebedard/kilted
May 1, 2025
Merged

Add support for Kilted#804
christophebedard merged 3 commits intomasterfrom
christophebedard/kilted

Conversation

@christophebedard
Copy link
Copy Markdown
Member

@christophebedard christophebedard commented May 1, 2025

Similar to #673, which did the same for Jazzy.

We need to use the testing apt repo to get Kilted binaries for tests before Kilted is officially released. See: https://docs.ros.org/en/kilted/Installation/Ubuntu-Install-Debians.html#enable-required-repositories. Once Kilted has been released, we can revert this.

Users will need to use use-ros2-testing: true for Kilted. On Windows, use the pre-release binaries. Also, allow using the testing repo for RHEL.

I won't mention Kilted in the README until it's officially been released

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@christophebedard christophebedard requested a review from a team as a code owner May 1, 2025 16:11
@christophebedard christophebedard requested review from MichaelOrlov and emersonknapp and removed request for a team May 1, 2025 16:11
Copy link
Copy Markdown
Member

@MichaelOrlov MichaelOrlov left a comment

Choose a reason for hiding this comment

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

LGTM. Please consider removing iron from supported distros as well.

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@christophebedard christophebedard merged commit c3b81c4 into master May 1, 2025
28 checks passed
@christophebedard christophebedard deleted the christophebedard/kilted branch May 1, 2025 18:36
@christophebedard
Copy link
Copy Markdown
Member Author

I will wait to test this with action-ros-ci before releasing it.

@christophebedard
Copy link
Copy Markdown
Member Author

Released as 0.7.12/v0.7.

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.

2 participants