-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Remove the line numbering in examples. #1906
Remove the line numbering in examples. #1906
Conversation
This is inconsistent with having the boxes be cut-n-paste. We also have to rewrite a small amount of text around it. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you anticipate reverting these changes once the upstream patch to Read-the-docs is released? Personally I would favor the previous style - with line numbers, and references to line numbers in-text - but of course that's broken now.
I don't anticipate reverting it only for these tutorials. That is, I'd like to keep the style of the tutorials consistent, and none of the other tutorials use line numbers. If we decide to implement line numbers for all tutorials, then that would be fine. |
Thanks for the reviews! I'm going to go ahead and merge this one, we can always make another change later on if needed. |
@Mergifyio backport galactic foxy |
This is inconsistent with having the boxes be cut-n-paste. We also have to rewrite a small amount of text around it. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit 35c4367)
This is inconsistent with having the boxes be cut-n-paste. We also have to rewrite a small amount of text around it. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit 35c4367)
Command
|
This is inconsistent with having the boxes be cut-n-paste. We also have to rewrite a small amount of text around it. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit 35c4367) Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
This is inconsistent with having the boxes be cut-n-paste. We also have to rewrite a small amount of text around it. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org> (cherry picked from commit 35c4367) Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
This is inconsistent with having the boxes be cut-n-paste.
We also have to rewrite a small amount of text around it.
Signed-off-by: Chris Lalancette clalancette@openrobotics.org
This should fix #1885