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 free fleet documentation #114

Merged
merged 5 commits into from
Jan 20, 2025
Merged

Update free fleet documentation #114

merged 5 commits into from
Jan 20, 2025

Conversation

aaronchongth
Copy link
Contributor

No description provided.

Copy link
Member

@mxgrey mxgrey 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 remembering to update this, it's very easy to overlook!

# Free Fleet
# Legacy - Free Fleet

<div class='warning'>
Copy link
Member

Choose a reason for hiding this comment

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

This warning class div doesn't have any effect in GitHub's markdown preview, but I've confirmed that it does render correctly in the fully generated mdbook 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh yeah, I realized the usual github markdown warning didn't work, but thankfully mdbook has some docs about it

## ROS 1 Navigation Stack

<div class='warning'>
The `zenoh-bridge-ros1` is an experimental bridge, and is actively being iterated on. Especially with <a href="https://wiki.ros.org/Distributions">ROS 1 going EOL in May 2025</a>, users are encouraged to migrate to ROS 2.
Copy link
Member

Choose a reason for hiding this comment

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

The ticks around zenoh-bridge-ros1 don't render correctly in the final mdbook when used inside the div. Not a big deal, but I thought I'd mention it. The name isn't ticked elsewhere so we could just remove the ticks here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, thanks for catching that. Removed ticks in 9b41750

@aaronchongth aaronchongth merged commit c26aa90 into master Jan 20, 2025
1 check passed
@aaronchongth aaronchongth deleted the ac/ff-efc-docs branch January 20, 2025 02:06
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