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

demo gdc 2025 #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

demo gdc 2025 #4

wants to merge 3 commits into from

Conversation

bruno-garcia
Copy link

@bruno-garcia bruno-garcia commented Mar 16, 2025

Profiling for the backend is off. Since the product doesn't pick a relevant thread by default and it's not great to have to pick one.
And since it's all sync, not super insightful as-is. We'd need to improve the demo to make it sense.

Having Blazor + JS on the frontend while both are disconnected (trace not connected) is not ideal. So we might be better off removing the Blazor tracing altogether.

TODO:

  1. Set up stack trace linking
  2. Make sure source context work
  3. Automate version bumping (right now hard coded to 1.0.25, since GDC 25)
  • While .NET has automatic versionining, JS wouldn't have any. Ideally we automate a version that aligns across the layers
  • Adjust release since sentry-cli sends the automatic one as it's unaware of whats given via Init

@bruno-garcia
Copy link
Author

/usr/lib/dotnet/sdk/8.0.112/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.TargetFrameworkInference.targets(166,5): error NETSDK1045: The current .NET SDK does not support targeting .NET 9.0.  Either target .NET 8.0 or lower, or use a version of the .NET SDK that supports .NET 9.0. Download the .NET SDK from https://aka.ms/dotnet/download [/home/runner/work/sentaur-leaderboard/sentaur-leaderboard/Sentaur.Leaderboard.Api/Sentaur.Leaderboard.Api.csproj]

Odd, only on ubuntu-latest doesn't have .NET 9? It's been out since Nov 2024

@bruno-garcia
Copy link
Author

bruno-garcia commented Mar 16, 2025

The latest release of .NET 9 is currently incompatible with Ubuntu and will not be included for the time being. For more details, please refer to the documentation. We are closing this issue, Thank you.

actions/runner-images#10964 (comment)

oof

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.

2 participants