Skip to content

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Oct 14, 2025

Fixes #36216

Pavel ... In addition to the new command example, can we also specify the Downloads folder for Windows because that's where the .nettrace files go (usually, I guess)?

... and all entries in the .nettrace files that I'm generating only show meterNames of System.Runtime.


Internal previews

📄 File 🔗 Preview link
.github/workflows/blazor-issue-processing.yml .github/workflows/blazor-issue-processing
aspnetcore/blazor/performance/webassembly-event-pipe-diagnostics.md aspnetcore/blazor/performance/webassembly-event-pipe-diagnostics

Updated the date and improved clarity in diagnostics instructions.
@guardrex guardrex self-assigned this Oct 14, 2025
@guardrex guardrex requested a review from Copilot October 14, 2025 13:55
Copy link
Contributor

@Copilot 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 Blazor WebAssembly Event Pipe diagnostics documentation to improve developer experience and best practices. The changes address issue #36216 by adding MSBuild conditions to restrict diagnostic features to Debug builds and providing clearer guidance on file locations.

Key changes:

  • Added MSBuild conditions to all diagnostic PropertyGroup elements to limit diagnostics to Debug builds only
  • Specified that .nettrace files are typically downloaded to the Downloads folder on Windows
  • Enhanced the metrics collection section with an example showing how to collect custom lifecycle meter data
Comments suppressed due to low confidence (1)

aspnetcore/blazor/performance/webassembly-event-pipe-diagnostics.md:89

  • Corrected spelling of 'devoloper' to 'developer'.
Call the preceding API from either a browser devoloper tools console or JavaScript code of the app.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Updated warnings and conditions for enabling diagnostics in production to clarify performance impacts.
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.

Add example to existing metric collection guidance

2 participants