getsentry / getsentry/sentry-dotnet

Add a benchmark action

Open
#3,332 0 comments 1 reaction 0 assignees View on GitHub
.NET Repository Maintenance Task
Dominant language
C#
Stars
770
Forks
248
Avg merge
3d 4h
Merged PRs (30d)
49

Description

> I've done away with the lambdas as well (these are now inline functions).
>
> The tests seem to pass now. 🎈

So indeed it seems like it has been the performance issue. Maybe we should check check that we have some benchmarks for these critical paths and introduce https://github.com/marketplace/actions/continuous-benchmark to the repo. Although it looks like it doesn't support PRs from outside contributors at the moment.

_Originally posted by @vaind in https://github.com/getsentry/sentry-dotnet/pull/3321#discussion_r1580533237_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.