Removed CanBeNull attribute

alinasmirnova · AndreyAkinshin · commit 7fbbc9f506ce · 2023-10-03T18:58:51.000+02:00
diff --git a/tests/BenchmarkDotNet.Tests/XUnit/EnvRequirementChecker.cs b/tests/BenchmarkDotNet.Tests/XUnit/EnvRequirementChecker.cs
@@ -9,11 +9,9 @@ namespace BenchmarkDotNet.Tests.XUnit;
 
 public static class EnvRequirementChecker
 {
-    [CanBeNull]
-    public static string GetSkip(params EnvRequirement[] requirements) => requirements.Select(GetSkip).FirstOrDefault(skip => skip != null);
+    public static string? GetSkip(params EnvRequirement[] requirements) => requirements.Select(GetSkip).FirstOrDefault(skip => skip != null);
 
-    [CanBeNull]
-    internal static string GetSkip(EnvRequirement requirement) => requirement switch
+    internal static string? GetSkip(EnvRequirement requirement) => requirement switch
     {
         EnvRequirement.WindowsOnly => RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? null : "Windows-only test",
         EnvRequirement.NonWindows => !RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? null : "Non-Windows test",

Original file line number	Diff line number	Diff line change
`@@ -9,11 +9,9 @@ namespace BenchmarkDotNet.Tests.XUnit;`
`9`	`9`
`10`	`10`	`public static class EnvRequirementChecker`
`11`	`11`	`{`
`12`		`- [CanBeNull]`
`13`		`- public static string GetSkip(params EnvRequirement[] requirements) => requirements.Select(GetSkip).FirstOrDefault(skip => skip != null);`
	`12`	`+ public static string? GetSkip(params EnvRequirement[] requirements) => requirements.Select(GetSkip).FirstOrDefault(skip => skip != null);`
`14`	`13`
`15`		`- [CanBeNull]`
`16`		`- internal static string GetSkip(EnvRequirement requirement) => requirement switch`
	`14`	`+ internal static string? GetSkip(EnvRequirement requirement) => requirement switch`
`17`	`15`	`{`
`18`	`16`	`EnvRequirement.WindowsOnly => RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? null : "Windows-only test",`
`19`	`17`	`EnvRequirement.NonWindows => !RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? null : "Non-Windows test",`