Skip to content

Conversation

@bjohansebas
Copy link
Member

Description

I’ve just discovered this page that has been helping me preview flame graphs, and it could be useful for others as well, avoiding the need to clone another package.

Validation

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

Updated instructions for generating and visualizing flame graphs, including browser preview and local setup options.

Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Copilot AI review requested due to automatic review settings January 15, 2026 16:40
@vercel
Copy link

vercel bot commented Jan 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
nodejs-org Ready Ready Preview Jan 19, 2026 5:38pm

Request Review

@github-actions
Copy link
Contributor

👋 Codeowner Review Request

The following codeowners have been identified for the changed files:

Team reviewers: @nodejs/diagnostics

Please review the changes when you have a chance. Thank you! 🙏

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the flame graphs documentation to provide users with an additional browser-based option for visualizing flame graphs using flamegraph.com, eliminating the need to clone and set up the FlameGraph tools locally.

Changes:

  • Added browser-based preview option using flamegraph.com website
  • Restructured step 6 to present two alternatives: browser preview and local FlameGraph tools setup
  • Updated the follow-up text to be more generic and apply to both visualization methods

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.97%. Comparing base (7a949e8) to head (57d0e5b).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8555      +/-   ##
==========================================
+ Coverage   74.96%   74.97%   +0.01%     
==========================================
  Files         103      103              
  Lines        9036     9036              
  Branches      312      312              
==========================================
+ Hits         6774     6775       +1     
+ Misses       2260     2259       -1     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 15, 2026

📦 Build Size Comparison

Summary

Metric Value
Old Total Size 3.74 MB
New Total Size 3.74 MB
Delta 0 B (0.00%)

Changes

➕ Added Assets (1)
Name Size
.next/static/chunks/3dcf6ad0c4617e37.js 205.34 KB
➖ Removed Assets (1)
Name Size
.next/static/chunks/f6056b949cae841a.js 205.34 KB

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Copy link
Member

@avivkeller avivkeller left a comment

Choose a reason for hiding this comment

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

Please lint

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.

4 participants