Skip to content

Add nanobind-dev key for more operating systems - #53562

Merged
mjcarroll merged 2 commits into
ros:masterfrom
sea-bass:nanobind-for-the-people
Aug 30, 2026
Merged

Add nanobind-dev key for more operating systems#53562
mjcarroll merged 2 commits into
ros:masterfrom
sea-bass:nanobind-for-the-people

Conversation

@sea-bass

Copy link
Copy Markdown
Contributor

@sea-bass
sea-bass requested a review from a team as a code owner August 29, 2026 01:40
@github-actions github-actions Bot added the rosdep Issue/PR is for a rosdep key label Aug 29, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ Platforms for new rosdep rules are valid

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For changes related to rosdep:

  • ✅ Platforms for new rosdep rules are valid
  • ✅ Installers for new rosdep rules are valid

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

@christophebedard christophebedard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@christophebedard christophebedard added the rosdep-needs-second-approval rosdep key PR is waiting for a 2nd approval label Aug 29, 2026
Comment thread rosdep/base.yaml
@mjcarroll
mjcarroll merged commit 6872637 into ros:master Aug 30, 2026
4 checks passed
@sea-bass
sea-bass deleted the nanobind-for-the-people branch August 30, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rosdep Issue/PR is for a rosdep key rosdep-needs-second-approval rosdep key PR is waiting for a 2nd approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants