Conversation
There was a problem hiding this comment.
Pull request overview
This PR replaces a global .editorconfig rule that suppressed all analyzer diagnostics for files under **/Migrations/**.cs with per-directory .editorconfig files in three Demo project migration directories. The new approach uses generated_code = true to mark migration files as generated code, which is the recommended Roslyn/Microsoft approach for suppressing analyzers on auto-generated files.
Changes:
- Removed the global
[**/Migrations/**.cs]section from the root.editorconfigthat disabled analyzer diagnostics for migration files. - Added
.editorconfigfiles withroot = trueandgenerated_code = trueto three Demo migration directories (Billing, Catalog, Shipping).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
.editorconfig |
Removed the global migration file analyzer suppression rule |
src/Mocha/src/Demo/Demo.Shipping/Migrations/.editorconfig |
New nested .editorconfig marking migration files as generated code |
src/Mocha/src/Demo/Demo.Catalog/Migrations/.editorconfig |
New nested .editorconfig marking migration files as generated code |
src/Mocha/src/Demo/Demo.Billing/Migrations/.editorconfig |
New nested .editorconfig marking migration files as generated code |
💡 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 @@
## main #9346 +/- ##
==========================================
- Coverage 74.16% 0 -74.17%
==========================================
Files 2677 0 -2677
Lines 140790 0 -140790
Branches 16371 0 -16371
==========================================
- Hits 104421 0 -104421
+ Misses 30774 0 -30774
+ Partials 5595 0 -5595
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:
|
Summary of the changes (Less than 80 chars)