Skip to content

Minor performance improvements on hot code paths#1947

Merged
bkoelman merged 6 commits intomasterfrom
perf
Mar 8, 2026
Merged

Minor performance improvements on hot code paths#1947
bkoelman merged 6 commits intomasterfrom
perf

Conversation

@bkoelman
Copy link
Member

@bkoelman bkoelman commented Mar 8, 2026

QUALITY CHECKLIST

  • Changes implemented in code
  • Complies with our contributing guidelines
  • N/A: Adapted tests
  • N/A: Documentation updated

@codecov
Copy link

codecov bot commented Mar 8, 2026

Codecov Report

❌ Patch coverage is 88.88889% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.19%. Comparing base (c0faae3) to head (5fde484).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...otNetCore/Queries/Parsing/QueryExpressionParser.cs 77.77% 1 Missing and 1 partial ⚠️
...DotNetCore/Serialization/Response/JsonApiWriter.cs 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1947      +/-   ##
==========================================
- Coverage   92.21%   92.19%   -0.03%     
==========================================
  Files         437      437              
  Lines       14927    14925       -2     
  Branches     2474     2476       +2     
==========================================
- Hits        13765    13760       -5     
- Misses        714      715       +1     
- Partials      448      450       +2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bkoelman bkoelman marked this pull request as ready for review March 8, 2026 18:55
@bkoelman bkoelman merged commit 21cc6e2 into master Mar 8, 2026
15 checks passed
@bkoelman bkoelman deleted the perf branch March 8, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant