Remove <autogenerated/> to enable CSharpier formatting
#42
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes the <auto-generated/> comment tags from all polyfill code files to enable CSharpier formatting during development. The change reverts PR #17 which had added these tags to instruct analyzers to skip the files, but inadvertently prevented CSharpier from formatting them.
Key changes:
- Removed 3-line auto-generated comment blocks from the beginning of all polyfill files
- Applied CSharpier formatting improvements including better line wrapping, consistent trailing commas, and improved spacing
- No functional code changes
Reviewed Changes
Copilot reviewed 107 out of 107 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| PolyShim/PolyfillExtensions.cs | Removed auto-generated comment block |
| PolyShim/NetCore30/*.cs (14 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/NetCore21/*.cs (8 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/NetCore20/*.cs (17 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/NetCore10/*.cs (14 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/Net90/*.cs (6 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/Net80/CancellationTokenSource.cs | Removed auto-generated comment block |
| PolyShim/Net70/*.cs (10 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/Net60/*.cs (13 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/Net50/*.cs (20 files) | Removed auto-generated comments and applied formatting improvements |
| PolyShim/NamespaceDummies.cs | Removed auto-generated comment block |
💡 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 #42 +/- ##
============================
============================
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Pull request #17 (@bfriesen) introduced the
<autogenerated/>tags at the beginning of each polyfill code file. Unfortunately, it also directs CSharpier to ignore these files during development. Reverting that PR for now until we find a way that satisfies both use cases.