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

Merge master into develop #769

Merged
merged 5 commits into from
Jun 8, 2023
Merged

Merge master into develop #769

merged 5 commits into from
Jun 8, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 8, 2023

Synchronizes changes of master branch into develop branch.

Conflicts in all launch files and changelogs.

* launch files

* changelogs

* restore supervisor for clock topic

* update universal robot

* Update CHANGELOG.rst
@omichel omichel requested a review from ygoumaz June 8, 2023 07:30
@ygoumaz ygoumaz added the synchronization Used when synchronizing branches label Jun 8, 2023
@ygoumaz ygoumaz self-assigned this Jun 8, 2023
@ygoumaz ygoumaz requested review from omichel and removed request for omichel June 8, 2023 12:53
@ygoumaz ygoumaz merged commit 3762d7a into develop Jun 8, 2023
@ygoumaz ygoumaz deleted the sync-master-720cb0c branch June 8, 2023 12:57
Comment on lines +41 to +42
version_package = version.parse(ET.parse(package_xml_path).findall("version")[0].text)
state_topic = '/state' if version_package < version.parse('3.7.0') else '/controller_state'
Copy link
Member

@omichel omichel Jun 9, 2023

Choose a reason for hiding this comment

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

That sounds like better English:

Suggested change
version_package = version.parse(ET.parse(package_xml_path).findall("version")[0].text)
state_topic = '/state' if version_package < version.parse('3.7.0') else '/controller_state'
package_version = version.parse(ET.parse(package_xml_path).findall("version")[0].text)
state_topic = '/state' if package_version < version.parse('3.7.0') else '/controller_state'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
synchronization Used when synchronizing branches
Development

Successfully merging this pull request may close these issues.

2 participants