-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Support Iron distro #2787
Support Iron distro #2787
Conversation
dfa9899
to
79956fe
Compare
.github/workflows/pre-release.yml
Outdated
@@ -31,12 +31,10 @@ jobs: | |||
strategy: | |||
fail-fast: false | |||
matrix: | |||
ros_distro: [humble, foxy] | |||
ros_distro: [humble] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will add iron here only when it builds and be ready on the ROS2 public package index
meanwhile, if we add it here, it will fail, since it will not find librealsense2 package for iron.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I say let's add it and ignore Iron failure, it should be fixed once it's merged , right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
update readme
Tracked by RSDEV-353