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

Setting the Client_Key of MicroXRCE-Agent #29

Merged
merged 3 commits into from Sep 16, 2019

Conversation

taro83
Copy link
Contributor

@taro83 taro83 commented Sep 9, 2019

Added an argument in Node() that can set Client Key of Micro XRCE-Agent at node initialization

Added an argument in Node() that can set Client Key of Micro XRCE-Agent at node initialization
@OpusK
Copy link
Contributor

OpusK commented Sep 10, 2019

Hi, @taro83

Thank you for your contribution.
I'm just leaving a few reviews, can you fix it?

@OpusK OpusK self-assigned this Sep 10, 2019
@OpusK OpusK added enhancement New feature or request in progress labels Sep 10, 2019
@OpusK
Copy link
Contributor

OpusK commented Sep 10, 2019

Please delete the files below and add them to .gitignore if necessary.

  • .DS_Store
  • src/.DS_Store

I edited ".gitignore" to ignore file named “.DS_Store”.
And I edited  "Node()" in ros2.hpp.
@taro83
Copy link
Contributor Author

taro83 commented Sep 10, 2019

Hi @OpusK
Thank you for your comment.
I added setting to ignore .DS_Store.

I will try to fix any other suggestions from you.

@OpusK
Copy link
Contributor

OpusK commented Sep 11, 2019

@taro83

You added .gitignore but didn't delete the DS_Store files.
Please delete it.

I delete ds_store
@taro83
Copy link
Contributor Author

taro83 commented Sep 11, 2019

@OpusK
Sorry, my mistake.
I deleted DS_Store files.

@OpusK
Copy link
Contributor

OpusK commented Sep 16, 2019

@taro83

Thanks for your contribution.
We will release this soon.

@OpusK OpusK merged commit d719691 into ROBOTIS-GIT:master Sep 16, 2019
@taro83
Copy link
Contributor Author

taro83 commented Sep 16, 2019

Thank you for merge !
I will follow this good project.

src/ros2/ros2.hpp Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants