Skip to content

Conversation

@jlblancoc
Copy link
Contributor

Increasing version of package(s) in repository mp2p_icp to 2.4.0-1:

mp2p_icp

* Merge pull request #27 <https://github.com/MOLAorg/mp2p_icp/issues/27> from MOLAorg/feat/new-filter-voxel-sor
* Add new unit test for class factory
* Add new FilterVoxelSOR filter
* Merge pull request #26 <https://github.com/MOLAorg/mp2p_icp/issues/26> from MOLAorg/feat/mm2las
* Add mm2las CLI tool
* Contributors: Jose Luis Blanco-Claraco

@jlblancoc jlblancoc requested a review from Yadunund as a code owner January 21, 2026 12:01
@github-actions github-actions bot added the rolling Issue/PR is for the ROS 2 Rolling distribution label Jan 21, 2026
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for sending a pull request to ROS distro!

This is an automated tool that helps check your pull request for correctness.
This tool checks a number of attributes associated with your ROS package and generates a report that helps our reviewers merge your pull request in a timely fashion. Here are a few things to consider when sending adding or updating a package to ROS Distro.
ROS Distro includes a very helpful CONTRIBUTING.md file that we recommend reading if it is your first time submitting a package.
Please also read the ROS Distro review guidelines which summarizes this release process.

ROS Distro Considerations

Package Considerations

Having your package included in a ROS Distro is a badge of quality, and we recommend that package developers strive to create packages of the highest quality. We recommend package developers review the following resources before submitting their package.

Need Help?

Please post your questions to Robotics Stack Exchange or refer to the Infrastructure General channel on our Zulip server.


For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

@ahcorde ahcorde added the held for sync Issue/PR has been held because the distribution is in a sync hold label Jan 21, 2026
@Yadunund
Copy link
Member

@jlblancoc kindly resolve merge conflicts

@christophebedard christophebedard added changes requested Maintainers have asked for changes to the pull request and removed held for sync Issue/PR has been held because the distribution is in a sync hold labels Jan 22, 2026
@marcoag
Copy link
Contributor

marcoag commented Jan 28, 2026

I'll close as it seems it has been superseded by #49619.

@jlblancoc feel free to reopen if needed.

@marcoag marcoag closed this Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changes requested Maintainers have asked for changes to the pull request rolling Issue/PR is for the ROS 2 Rolling distribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants