dotnet / nuget-trends

Check out NuGet packages adoption and what's trending on NuGet.
https://nugettrends.com
MIT License
144 stars 25 forks source link

sentry performance monitoring and source context #219

Closed bruno-garcia closed 1 year ago

bruno-garcia commented 1 year ago
  1. Gets additional context on Angular's performance
  2. Line numbers for framework frames, and source context for C# stack traces for the app
codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (5ce3ac8) 93.24% compared to head (785b874) 93.24%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #219 +/- ## ======================================= Coverage 93.24% 93.24% ======================================= Files 27 27 Lines 429 429 Branches 33 33 ======================================= Hits 400 400 Misses 27 27 Partials 2 2 ``` | [Impacted Files](https://codecov.io/gh/dotnet/nuget-trends/pull/219?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | Coverage Δ | | |---|---|---| | [....Web/Portal/src/app/packages/packages.component.ts](https://codecov.io/gh/dotnet/nuget-trends/pull/219?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-c3JjL051R2V0VHJlbmRzLldlYi9Qb3J0YWwvc3JjL2FwcC9wYWNrYWdlcy9wYWNrYWdlcy5jb21wb25lbnQudHM=) | `94.89% <ø> (ø)` | | | [...s/loading-indicator/loading-indicator.component.ts](https://codecov.io/gh/dotnet/nuget-trends/pull/219?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-c3JjL051R2V0VHJlbmRzLldlYi9Qb3J0YWwvc3JjL2FwcC9zaGFyZWQvY29tcG9uZW50cy9sb2FkaW5nLWluZGljYXRvci9sb2FkaW5nLWluZGljYXRvci5jb21wb25lbnQudHM=) | `100.00% <ø> (ø)` | | | [.../components/package-list/package-list.component.ts](https://codecov.io/gh/dotnet/nuget-trends/pull/219?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-c3JjL051R2V0VHJlbmRzLldlYi9Qb3J0YWwvc3JjL2FwcC9zaGFyZWQvY29tcG9uZW50cy9wYWNrYWdlLWxpc3QvcGFja2FnZS1saXN0LmNvbXBvbmVudC50cw==) | `100.00% <ø> (ø)` | | | [.../components/search-input/search-input.component.ts](https://codecov.io/gh/dotnet/nuget-trends/pull/219?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-c3JjL051R2V0VHJlbmRzLldlYi9Qb3J0YWwvc3JjL2FwcC9zaGFyZWQvY29tcG9uZW50cy9zZWFyY2gtaW5wdXQvc2VhcmNoLWlucHV0LmNvbXBvbmVudC50cw==) | `90.62% <ø> (ø)` | | | [...omponents/share-popover/share-popover.component.ts](https://codecov.io/gh/dotnet/nuget-trends/pull/219?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-c3JjL051R2V0VHJlbmRzLldlYi9Qb3J0YWwvc3JjL2FwcC9zaGFyZWQvY29tcG9uZW50cy9zaGFyZS1wb3BvdmVyL3NoYXJlLXBvcG92ZXIuY29tcG9uZW50LnRz) | `100.00% <ø> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

bruno-garcia commented 1 year ago

Symbol upload failed with:

  NuGetTrends.Web -> /home/runner/work/nuget-trends/nuget-trends/src/NuGetTrends.Web/bin/Release/net7.0/NuGetTrends.Web.dll
  Preparing to upload debug symbols to Sentry for NuGetTrends.Web (Release/net7.0)
  > Found 1 debug information file
  > Prepared debug information file for upload
EXEC : warning : API request failed [/home/runner/work/nuget-trends/nuget-trends/src/NuGetTrends.Web/NuGetTrends.Web.csproj]
caused by : sentry reported an warning : The requested resource does not exist (http status: 404) [/home/runner/work/nuget-trends/nuget-trends/src/NuGetTrends.Web/NuGetTrends.Web.csproj]

  Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.
  Please attach the full debug log to all bug reports.
/home/runner/.nuget/packages/sentry/3.29.1/buildTransitive/Sentry.targets(135,5): warning : Sentry CLI could not upload debug files. [/home/runner/work/nuget-trends/nuget-trends/src/NuGetTrends.Web/NuGetTrends.Web.csproj]

Build succeeded.

EXEC : warning : API request failed [/home/runner/work/nuget-trends/nuget-trends/src/NuGetTrends.Web/NuGetTrends.Web.csproj]
caused by : sentry reported an warning : The requested resource does not exist (http status: 404) [/home/runner/work/nuget-trends/nuget-trends/src/NuGetTrends.Web/NuGetTrends.Web.csproj]

Wrong org name