Skip to content
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

dynamixel_hardware: 0.3.1-1 in 'rolling/distribution.yaml' [bloom] #35400

Merged
merged 2 commits into from
Dec 1, 2022

Conversation

youtalk
Copy link
Contributor

@youtalk youtalk commented Nov 23, 2022

Increasing version of package(s) in repository dynamixel_hardware to 0.3.1-1:

dynamixel_hardware

* Merge pull request #39 <https://github.com/youtalk/dynamixel_control/issues/39> from ijnek/ijnek-unused-parameters
* Merge pull request #31 <https://github.com/youtalk/dynamixel_control/issues/31> from ijnek/ijnek-unused-parameter-2
* Merge pull request #25 <https://github.com/youtalk/dynamixel_control/issues/25> from ijnek/ijnek-new-hardware-interface
* Merge pull request #24 <https://github.com/youtalk/dynamixel_control/issues/24> from ijnek/ijnek-add-callbackreturn-reference
* Merge pull request #20 <https://github.com/youtalk/dynamixel_control/issues/20> from pdenes/add-extra-params
* Merge pull request #19 <https://github.com/youtalk/dynamixel_control/issues/19> from pdenes/add-dependencies
* Merge pull request #17 <https://github.com/youtalk/dynamixel_control/issues/17> from youtalk/galactic
* Merge pull request #16 <https://github.com/youtalk/dynamixel_control/issues/16> from Schnilz/main
* Merge pull request #13 <https://github.com/youtalk/dynamixel_control/issues/13> from youtalk/change-loglevel
* Merge pull request #10 <https://github.com/youtalk/dynamixel_control/issues/10> from youtalk/joint-id
* Merge pull request #1 <https://github.com/youtalk/dynamixel_control/issues/1> from youtalk/velocity-control
* Contributors: Kenji Brameld, Nils Schulte, Pal Denes, Yutaka Kondo

@github-actions github-actions bot added the rolling Issue/PR is for the ROS 2 Rolling distribution label Nov 23, 2022
release:
tags:
release: release/rolling/{package}/{version}
url: https://github.com/dynamixel-community/dynamixel_hardware-release.git
Copy link
Contributor

Choose a reason for hiding this comment

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

Our best practices here are to use an https://github.com/ros2-gbp repository for releases into Rolling. That way, we'll be able to automatically release this package from Rolling into the next ROS distribution (Iron). If you'd like to do this, then please see https://github.com/ros2-gbp/ros2-gbp-github-org/blob/latest/CONTRIBUTING.md on how to create one.

Copy link
Member

Choose a reason for hiding this comment

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

I have opened ros2-gbp/ros2-gbp-github-org#160 to move the release repository.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you very much for your help. I'd like to transfer the repository.

Copy link
Member

Choose a reason for hiding this comment

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

I've mirrored the current repository and the permissions updates are deploying now.

nuclearsandwich added a commit to ros2-gbp/ros2-gbp-github-org that referenced this pull request Nov 29, 2022
nuclearsandwich added a commit to ros2-gbp/ros2-gbp-github-org that referenced this pull request Nov 30, 2022
Copy link
Member

@nuclearsandwich nuclearsandwich left a comment

Choose a reason for hiding this comment

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

Once CI confirms that the latest changes are sound this can merge.

@nuclearsandwich nuclearsandwich merged commit 89df445 into ros:master Dec 1, 2022
@youtalk youtalk deleted the bloom-dynamixel_hardware-0 branch December 2, 2022 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rolling Issue/PR is for the ROS 2 Rolling distribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants