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

Update: crate to bluerobotics-ping #27

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

RaulTrombin
Copy link
Member

ping-rs is already in use.

@RaulTrombin RaulTrombin requested review from patrickelectric and joaoantoniocardoso and removed request for patrickelectric April 24, 2024 17:34
@RaulTrombin RaulTrombin force-pushed the create_crate branch 3 times, most recently from f11e70e to 285e2b4 Compare April 24, 2024 17:51
@patrickelectric
Copy link
Member

This PR is in conflict

@RaulTrombin
Copy link
Member Author

This PR is in conflict

rebased

@@ -1,2 +1,2 @@
# ping-rs
https://docs.bluerobotics.com/ping-rs/ping_rs/
https://docs.bluerobotics.com/ping-rs/bluerobotics_ping/
Copy link
Member

Choose a reason for hiding this comment

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

This is not the name of the repository, we also don´t intend to change since this matches with all others ping repositories that we currently have already, it's ok to have the crate with a difference name though.

README.md Outdated
@@ -1,2 +1,2 @@
# ping-rs
https://docs.bluerobotics.com/ping-rs/ping_rs/
https://docs.bluerobotics.com/ping-rs/ping-rs/
Copy link
Member

Choose a reason for hiding this comment

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

The first one is the repository name and the second is the crate name, since you changed the crate name the link should also take that

@patrickelectric patrickelectric merged commit 1825fdc into bluerobotics:master Apr 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants