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

Fix Navigation Issues in Docs #2123

Closed
wants to merge 4 commits into from

Conversation

kernel-patches-bot
Copy link

Pull request for series with
subject: Fix Navigation Issues in Docs
version: 1
url: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421

@kernel-patches-bot
Copy link
Author

Master branch: 7cc595a
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: 325d956
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: 7cc595a
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: 325d956
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: e504389
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: a5bdc36
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: e47d0bf
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: 4746158
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: e12cd15
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: ebf7f6f
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

Master branch: d41bc48
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

Nobody and others added 4 commits November 16, 2021 21:21
This changes the type of underline used to follow the guidelines in
Documentation/doc-guide/sphinx.rst which also ensures that the headings
are rendered at the correct level in the HTML sidebar

Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
This allows for documentation relating to BPF Program Types to be
matched by the glob pattern prog_* for inclusion in a sphinx toctree

Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
Acked-by: KP Singh <kpsingh@kernel.org>
This commit fixes the display of the BPF documentation in the sidebar
when rendered as HTML.

Before this patch, the sidebar would render as follows for some
sections:

| BPF Documentation
  |- BPF Type Format (BTF)
    |- BPF Type Format (BTF)

This was due to creating a heading in index.rst followed by
a sphinx toctree, where the file referenced carries the same
title as the section heading.

To fix this I applied a pattern that has been established in other
subfolders of Documentation:

1. Re-wrote index.rst to have a single toctree
2. Split the sections out in to their own files

Additionally maps.rst and programs.rst make use of a glob pattern to
include map_* or prog_* rst files in their toctree, meaning future map
or program type documentation will be automatically included.

Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
@kernel-patches-bot
Copy link
Author

Master branch: db813d7
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=579421
version: 1

@kernel-patches-bot
Copy link
Author

At least one diff in series https://patchwork.kernel.org/project/netdevbpf/list/?series=579421 irrelevant now. Closing PR.

@kernel-patches-bot kernel-patches-bot deleted the series/579421=>bpf-next branch November 17, 2021 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants