-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Clarify compatibility #3694
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
Clarify compatibility #3694
Conversation
- Add ASP.NET Core versions. - Clarify that Microsoft.AspNetCore.OpenApi is the only part of ASP.NET Core dependent on Microsoft.OpenApi. - Remove v7 from the compatibility table as 6.0 is now out-of-support. - Add link to SemVer website.
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.
Pull request overview
This PR updates the compatibility documentation to reflect current ASP.NET Core version support and clarifies dependency relationships. The changes remove outdated information about ASP.NET Core 6.0 (now out-of-support) and improve the accuracy of technical explanations.
- Added a link to the Semantic Versioning website for better context
- Clarified that only the Microsoft.AspNetCore.OpenApi NuGet package has a dependency on Microsoft.OpenApi v2+
- Removed v7 compatibility information from the table as ASP.NET Core 6.0 is no longer supported
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/migrating-to-v10.md | Added SemVer link reference and clarified that Microsoft.AspNetCore.OpenApi package specifically depends on Microsoft.OpenApi v2+ |
| README.md | Removed v7 compatibility row and updated ASP.NET Core version specifications from >= 8.0.0 to explicit version ranges (9.0.x, 8.0.x) for v9 and v8 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3694 +/- ##
=======================================
Coverage 94.66% 94.66%
=======================================
Files 111 111
Lines 3858 3858
Branches 778 778
=======================================
Hits 3652 3652
Misses 206 206
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Resolves #3693.