Skip to content

chore: remove cyclonedds repo in release_script and repos templete#244

Open
xygyo77 wants to merge 1 commit into
tier4:mainfrom
xygyo77:remove/cyclonedds-dependency
Open

chore: remove cyclonedds repo in release_script and repos templete#244
xygyo77 wants to merge 1 commit into
tier4:mainfrom
xygyo77:remove/cyclonedds-dependency

Conversation

@xygyo77

@xygyo77 xygyo77 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description

Due to improvements in hook functions, bundling CycloneDDS with CARET is no longer necessary. This PR removes the CycloneDDS meta-repository information from the .repos file generation process in the release script.

Related links

https://tier4.atlassian.net/browse/SYSPERF-141
https://tier4.atlassian.net/browse/SYSPERF-126

Notes for reviewers

Pre-review checklist for the PR author

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

  • The PR has been properly tested.
  • The PR has been reviewed.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • (Optional) The PR has been properly tested with CARET_report verification.
  • There are no open discussions or they are tracked via tickets.
  • The PR is ready for merge.

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

@xygyo77 xygyo77 requested a review from miyakoshi-dev July 3, 2026 05:38
@xygyo77 xygyo77 changed the title chore: remove cyclonedds repo in release_script and temporary repos chore: remove cyclonedds repo in release_script and repos templete Jul 3, 2026
miyakoshi-dev
miyakoshi-dev previously approved these changes Jul 7, 2026

@miyakoshi-dev miyakoshi-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@miyakoshi-dev miyakoshi-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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