Skip to content

Add trace logging for component schema generation #1709

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 30, 2025

Conversation

bkoelman
Copy link
Member

@bkoelman bkoelman commented Mar 30, 2025

Turned off by default, but is turned on in OpenApiTests.

QUALITY CHECKLIST

Fixes #1626.

Copy link

codecov bot commented Mar 30, 2025

Codecov Report

Attention: Patch coverage is 94.67456% with 9 lines in your changes missing coverage. Please review.

Project coverage is 90.79%. Comparing base (b5f0a9e) to head (632bb7b).
Report is 1 commits behind head on openapi.

Files with missing lines Patch % Lines
...Core.OpenApi.Swashbuckle/SchemaGenerationTracer.cs 82.69% 7 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           openapi    #1709      +/-   ##
===========================================
+ Coverage    90.75%   90.79%   +0.04%     
===========================================
  Files          466      467       +1     
  Lines        14451    14596     +145     
  Branches      2293     2296       +3     
===========================================
+ Hits         13115    13253     +138     
- Misses         909      915       +6     
- Partials       427      428       +1     

☔ 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 force-pushed the openapi-trace-logging branch from a7b41dd to 0ecfc54 Compare March 30, 2025 22:19
@bkoelman bkoelman force-pushed the openapi-trace-logging branch from 0ecfc54 to 632bb7b Compare March 30, 2025 23:07
@bkoelman bkoelman marked this pull request as ready for review March 30, 2025 23:27
@bkoelman bkoelman merged commit 87f8d52 into openapi Mar 30, 2025
16 checks passed
@bkoelman bkoelman deleted the openapi-trace-logging branch March 30, 2025 23:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant