Skip to content

Conversation

@YandongJi
Copy link
Contributor

No description provided.

Copy link
Member

@stephane-caron stephane-caron left a comment

Choose a reason for hiding this comment

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

Thank you for starting this PR 👍 Currently the tests won't pass because of the url issue (see suggestion comment).

Co-authored-by: Stéphane Caron <1189580+stephane-caron@users.noreply.github.com>
@stephane-caron
Copy link
Member

Thank you for following up 👍 The failing CI is caused by another issue unrelated to this PR. I will go ahead and merge this, then fix continuous integration in #187.

@stephane-caron stephane-caron merged commit 2eec699 into robot-descriptions:main Nov 17, 2025
7 of 15 checks passed
stephane-caron added a commit that referenced this pull request Nov 17, 2025
Unfortunately this description is not open source.
stephane-caron added a commit that referenced this pull request Nov 17, 2025
Unfortunately this description is not open source. The discussion is to
be continued in
#186.
@stephane-caron
Copy link
Member

My apologies for not checking carefully: actually we can't add this description yet because it is not covered by an open-source license; see Adding a new robot description in the contribution guidelines. I am temporarily reverting the addition in ec3804c until we verify all checks.

Can an open-source license be added upstream? Or is there an alternative repository for this robot description with an open-source license?

stephane-caron added a commit that referenced this pull request Nov 17, 2025
Unfortunately this description is not open source. The discussion is to
be continued in
#186.
stephane-caron added a commit that referenced this pull request Nov 17, 2025
Unfortunately this description is not open source. The discussion is to
be continued in
#186.
@stephane-caron stephane-caron changed the title add galaxea robot License missing: add galaxea robot Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants