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.

Sign up for GitHub

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

Jump to bottom

Fix up some warnings when building the documentation. #4261

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Fix up some warnings when building the documentation. #4261

merged 1 commit into from
Apr 4, 2024

Conversation

clalancette
Copy link
Contributor

  • We don't need a redirect for a new page.
  • We don't need an anchor for a new page.
  • The length of the underline should match the text above.
  • italics are done with double `` in rst, not single.

@clalancette
Fix up some warnings when building the documentation.
27fa4f8 
* We don't need a redirect for a new page.
* We don't need an anchor for a new page.
* The length of the underline should match the text above.
* italics are done with double `` in rst, not single.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette clalancette requested a review from audrow as a code owner April 4, 2024 18:43
@clalancette clalancette added the backport-all backport at reviewers discretion; from rolling to all versions label Apr 4, 2024
@clalancette
Copy link
Contributor Author

I'm just going to go ahead and merge this one to fix the problems here. If there are any follow-ups from reviewers, I'm happy to apply those separately.

@clalancette clalancette merged commit a567e19 into rolling Apr 4, 2024
4 checks passed
@delete-merged-branch delete-merged-branch bot deleted the clalancette/fix-urdf branch April 4, 2024 18:47
mergify bot pushed a commit that referenced this pull request Apr 4, 2024
@clalancette @mergify
Fix up some warnings when building the documentation. (#4261)
a3384e6 
* We don't need a redirect for a new page.
* We don't need an anchor for a new page.
* The length of the underline should match the text above.
* italics are done with double `` in rst, not single.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit a567e19)
@mergify mergify bot mentioned this pull request Apr 4, 2024
Merged
mergify bot pushed a commit that referenced this pull request Apr 4, 2024
@clalancette @mergify
Fix up some warnings when building the documentation. (#4261)
f50d926 
* We don't need a redirect for a new page.
* We don't need an anchor for a new page.
* The length of the underline should match the text above.
* italics are done with double `` in rst, not single.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit a567e19)
@mergify mergify bot mentioned this pull request Apr 4, 2024
Merged
clalancette added a commit that referenced this pull request Apr 4, 2024
@mergify @clalancette
Fix up some warnings when building the documentation. (#4261) (#4262)
d402e00 
* We don't need a redirect for a new page.
* We don't need an anchor for a new page.
* The length of the underline should match the text above.
* italics are done with double `` in rst, not single.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit a567e19)

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
clalancette added a commit that referenced this pull request Apr 4, 2024
@mergify @clalancette
Fix up some warnings when building the documentation. (#4261) (#4263)
20c08f1 
* We don't need a redirect for a new page.
* We don't need an anchor for a new page.
* The length of the underline should match the text above.
* italics are done with double `` in rst, not single.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit a567e19)

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@audrow audrow Awaiting requested review from audrow audrow is a code owner

Assignees
No one assigned
Labels
backport-all backport at reviewers discretion; from rolling to all versions
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@clalancette