powertools-lambda-dotnet icon indicating copy to clipboard operation
powertools-lambda-dotnet copied to clipboard

chore: Logging AOT Support WIP

Open hjgraca opened this issue 1 year ago • 2 comments

…fic paths. New helper method to convert anonymous types to dictionary, use for source generation .NET 8

Please provide the issue number

Issue number: #212

Summary

Changes

Logging support for native AOT .NET 8 specific serialization paths New helper method to convert anonymous types to dictionary, use for source generation .NET 8

User experience

Please share what the user experience looks like before and after this change

Checklist

Please leave checklist items unchecked if they do not apply to your change.

Is this a breaking change?

RFC issue number:

Checklist:

  • [ ] Migration process documented
  • [ ] Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

hjgraca avatar Aug 02 '24 10:08 hjgraca

Codecov Report

Attention: Patch coverage is 87.38899% with 71 lines in your changes missing coverage. Please review.

Project coverage is 72.94%. Comparing base (5a483b7) to head (e1d1181). Report is 43 commits behind head on develop.

Files with missing lines Patch % Lines
...ambda.Powertools.Logging/Internal/LoggingAspect.cs 74.82% 32 Missing and 4 partials :warning:
...Serializers/PowertoolsSourceGeneratorSerializer.cs 55.00% 7 Missing and 2 partials :warning:
...ging/Internal/PowertoolsConfigurationsExtension.cs 96.73% 1 Missing and 4 partials :warning:
.../Internal/Converters/ExceptionPropertyExtractor.cs 66.66% 4 Missing :warning:
...ogging/Internal/Helpers/PowertoolsLoggerHelpers.cs 76.47% 2 Missing and 2 partials :warning:
...raries/src/AWS.Lambda.Powertools.Logging/Logger.cs 63.63% 1 Missing and 3 partials :warning:
....Logging/Internal/Converters/ExceptionConverter.cs 50.00% 3 Missing :warning:
...da.Powertools.Logging/Internal/PowertoolsLogger.cs 96.00% 1 Missing and 2 partials :warning:
...Powertools.Common/Core/PowertoolsConfigurations.cs 50.00% 0 Missing and 1 partial :warning:
...owertools.Logging/Internal/LoggingLambdaContext.cs 97.14% 0 Missing and 1 partial :warning:
... and 1 more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #628      +/-   ##
===========================================
+ Coverage    72.71%   72.94%   +0.22%     
===========================================
  Files          190      195       +5     
  Lines         7886     7986     +100     
  Branches       850      854       +4     
===========================================
+ Hits          5734     5825      +91     
- Misses        1862     1876      +14     
+ Partials       290      285       -5     
Flag Coverage Δ
unittests 72.94% <87.38%> (+0.22%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Aug 02 '24 10:08 codecov[bot]

  • [x] documentation

hjgraca avatar Sep 18 '24 08:09 hjgraca