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

[DOC] Restructure / Refine new profiling docs #2794

Closed
5 tasks done
Rperry2174 opened this issue Dec 1, 2023 · 3 comments · Fixed by #2855
Closed
5 tasks done

[DOC] Restructure / Refine new profiling docs #2794

Rperry2174 opened this issue Dec 1, 2023 · 3 comments · Fixed by #2855
Assignees
Labels
type/docs Improvements for doc docs. Used by Docs team for project management

Comments

@Rperry2174
Copy link
Contributor

Rperry2174 commented Dec 1, 2023

In #2747 I merged things in a little bit of a "halfway" state.

@knylander-grafana had some recommendations about reorganizing but I didn't want the PR to go stale and also wanted to see what it looked like in the sidebar before reshuffling things around

image

The three files in question are

  • docs/sources/why-use-continuous-profiling/_index.md
  • docs/sources/analyzing-profiles-in-pyroscope-ui/_index.md
  • docs/sources/profiling-types-in-pyroscope/_index.md

Related:

Tasks

  1. type/docs
  2. type/docs
    Rperry2174 knylander-grafana
  3. type/docs
    Rperry2174 knylander-grafana
  4. type/docs
    knylander-grafana
@Rperry2174 Rperry2174 added the type/docs Improvements for doc docs. Used by Docs team for project management label Dec 1, 2023
@knylander-grafana
Copy link
Contributor

@Rperry2174 I'll tackle this issue during the sprint next week.

@knylander-grafana
Copy link
Contributor

Related: #2828

@knylander-grafana
Copy link
Contributor

PR for some of this content: #2839

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Improvements for doc docs. Used by Docs team for project management
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants