-
Notifications
You must be signed in to change notification settings - Fork 511
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
[net8.0] Merge main into net8.0. #18494
[net8.0] Merge main into net8.0. #18494
Conversation
There's no purpose to catching an exception just to throw it, so remove the entire try/catch handler. Fixes this compiler warning: src/bgen/Generator.cs(1284,4): warning CA2200: Re-throwing caught exception changes stack information
We don't support watchOS on .NET, so this is just useless code.
…rin#18474) This makes it easy to copy-paste the path to the binlog to open it in a binlog viewer.
…arin#17831) Added the following new checks: ```csharp // dotnet_style_coalesce_expression = true var v = x ?? y; // dotnet_style_coalesce_expression = false var v = x != null ? x : y; // or var v = x == null ? y : x; ``` ```csharp // dotnet_style_null_propagation = true var v = o?.ToString(); // dotnet_style_null_propagation = false var v = o == null ? null : o.ToString(); // or var v = o != null ? o.String() : null; ``` ```csharp // dotnet_style_prefer_is_null_check_over_reference_equality_method = true if (value is null) return; // dotnet_style_prefer_is_null_check_over_reference_equality_method = false if (object.ReferenceEquals(value, null)) return; // dotnet_style_prefer_is_null_check_over_reference_equality_method = false if ((object)o == null) return; ```
…ss. (xamarin#18456) Integrate class handle rewriting into static registrar.
💻 [CI Build] Windows Integration Tests passed 💻✅ All Windows Integration Tests passed. Pipeline on Agent |
❌ [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) failed ❌Failed tests are:
Pipeline on Agent |
📚 [PR Build] Artifacts 📚Packages generatedView packagesPipeline on Agent |
✅ API diff for current PR / commitNET (empty diffs)
✅ API diff vs stable.NET (No breaking changes)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
🔥 [CI Build] Test results 🔥Test results❌ Tests failed on VSTS: simulator tests 0 tests crashed, 1 tests failed, 91 tests passed. Failures❌ dotnettests tests [attempt 4]
Html Report (VSDrops) Download Successes
Pipeline on Agent |
Test failures are unrelated. |
No description provided.