- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.3k
Add .NET 8 support #2799
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
Add .NET 8 support #2799
Conversation
| Codecov ReportAll modified and coverable lines are covered by tests ✅ 
 ❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@            Coverage Diff             @@
##           master    #2799      +/-   ##
==========================================
- Coverage   92.42%   86.08%   -6.35%     
==========================================
  Files          94       73      -21     
  Lines        3183     2795     -388     
  Branches      556      449     -107     
==========================================
- Hits         2942     2406     -536     
- Misses        241      389     +148     
 Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. | 
9819a4f    to
    540b2dc      
    Compare
  
    75512e3    to
    28aaca1      
    Compare
  
    7b948e9    to
    f0fab28      
    Compare
  
    | { | ||
| #if (!NETSTANDARD2_0) | ||
| var serializerOptions = s.GetService<IOptions<JsonOptions>>()?.Value?.JsonSerializerOptions | ||
| var serializerOptions = | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be a good idea to document this order of options somewhere, because not documenting it will most likely raise questions about what options are used.
        
          
                ...ashbuckle.AspNetCore.SwaggerGen/DependencyInjection/SwaggerGenServiceCollectionExtensions.cs
              
                Outdated
          
            Show resolved
            Hide resolved
        
      - Add support for .NET 8. - Bump version to `6.6.0`.
Add WeatherForecast .NET 8 Minimal API test application. Resolves domaindrivendev#2791.
Add test for domaindrivendev#2771 for `DateOnly` values.
Use JsonOptions for Minimal APIs in .NET 8+ if not available from MVC. Use `JsonSerializerOptions.Default` where available. See domaindrivendev#2593 (comment).
Use the Regex source generator for .NET 7+. Resolves domaindrivendev#2794.
6.6.0.Resolves #2792
Resolves #2791
Resolves #2771
Resolves #2794