[UIKit] Enable nullability and clean up UICollectionViewLayoutAttributes.#24606
Conversation
…tes. * Enable nullability (#nullable enable). * Use ArgumentNullException.ThrowIfNull() instead of manual null checks. * Improve XML documentation comments: fix formatting (remove extra whitespace), fix order (summary first, then typeparam, param, returns, remarks), add 'see cref' references, fix incomplete sentences, use <c> for code references. Contributes towards #17285.
There was a problem hiding this comment.
Pull request overview
This PR enables nullable reference types and cleans up XML documentation in the UICollectionViewLayoutAttributes class to improve code quality and API documentation.
Changes:
- Enabled nullable reference types by adding
#nullable enable - Modernized null checks by replacing manual checks with
ArgumentNullException.ThrowIfNull() - Improved XML documentation formatting, ordering, and clarity with proper
see crefreferences
✅ [CI Build #cbcd3f9] Build passed (Build packages) ✅Pipeline on Agent |
✅ [PR Build #cbcd3f9] Build passed (Detect API changes) ✅Pipeline on Agent |
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
✅ [CI Build #cbcd3f9] Build passed (Build macOS tests) ✅Pipeline on Agent |
💻 [CI Build #cbcd3f9] Tests on macOS M1 - Mac Monterey (12) passed 💻✅ All tests on macOS M1 - Mac Monterey (12) passed. Pipeline on Agent |
💻 [CI Build #cbcd3f9] Tests on macOS X64 - Mac Sonoma (14) passed 💻✅ All tests on macOS X64 - Mac Sonoma (14) passed. Pipeline on Agent |
💻 [CI Build #cbcd3f9] Tests on macOS arm64 - Mac Sequoia (15) passed 💻✅ All tests on macOS arm64 - Mac Sequoia (15) passed. Pipeline on Agent |
💻 [CI Build #cbcd3f9] Tests on macOS M1 - Mac Ventura (13) passed 💻✅ All tests on macOS M1 - Mac Ventura (13) passed. Pipeline on Agent |
💻 [CI Build #cbcd3f9] Tests on macOS arm64 - Mac Tahoe (26) passed 💻✅ All tests on macOS arm64 - Mac Tahoe (26) passed. Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
🚀 [CI Build #cbcd3f9] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 117 tests passed 🎉 Tests counts✅ cecil: All 1 tests passed. Html Report (VSDrops) Download Pipeline on Agent |
This is file 6 of 30 files with nullability disabled in UIKit.
Contributes towards #17285.