Enable nullable annotations for Volo.Abp.RemoteServices

framework/src/Volo.Abp.RemoteServices/Volo.Abp.RemoteServices.csproj

@@ -5,6 +5,8 @@
 
     <PropertyGroup>
         <TargetFrameworks>netstandard2.0;netstandard2.1;net7.0</TargetFrameworks>
+        <Nullable>enable</Nullable>
+        <WarningsAsErrors>Nullable</WarningsAsErrors>
         <AssemblyName>Volo.Abp.RemoteServices</AssemblyName>
         <PackageId>Volo.Abp.RemoteServices</PackageId>
         <AssetTargetFallback>$(AssetTargetFallback);portable-net45+win8+wp8+wpa81;</AssetTargetFallback>

framework/src/Volo.Abp.RemoteServices/Volo/Abp/Http/Client/IRemoteServiceConfigurationProvider.cs

@@ -8,6 +8,5 @@ public interface IRemoteServiceConfigurationProvider
     [ItemNotNull]
     Task<RemoteServiceConfiguration> GetConfigurationOrDefaultAsync(string name);
 
-    [ItemCanBeNull]
+    Task<RemoteServiceConfiguration?> GetConfigurationOrDefaultOrNullAsync(string name);
-    Task<RemoteServiceConfiguration> GetConfigurationOrDefaultOrNullAsync(string name);
 }

framework/src/Volo.Abp.RemoteServices/Volo/Abp/Http/Client/RemoteServiceConfiguration.cs

@@ -2,12 +2,12 @@
 
 namespace Volo.Abp.Http.Client;
 
-public class RemoteServiceConfiguration : Dictionary<string, string>
+public class RemoteServiceConfiguration : Dictionary<string, string?>
 {
     /// <summary>
     /// Base Url.
     /// </summary>
-    public string BaseUrl {
+    public string? BaseUrl {
         get => this.GetOrDefault(nameof(BaseUrl));
         set => this[nameof(BaseUrl)] = value;
     }
@@ -15,7 +15,7 @@ public class RemoteServiceConfiguration : Dictionary<string, string>
     /// <summary>
     /// Version.
     /// </summary>
-    public string Version {
+    public string? Version {
         get => this.GetOrDefault(nameof(Version));
         set => this[nameof(Version)] = value;
     }
@@ -25,7 +25,7 @@ public class RemoteServiceConfiguration : Dictionary<string, string>
 
     }
 
-    public RemoteServiceConfiguration(string baseUrl, string version = null)
+    public RemoteServiceConfiguration(string baseUrl, string? version = null)
     {
         this[nameof(BaseUrl)] = baseUrl;
         this[nameof(Version)] = version;

framework/src/Volo.Abp.RemoteServices/Volo/Abp/Http/Client/RemoteServiceConfigurationDictionary.cs

@@ -3,11 +3,11 @@ using JetBrains.Annotations;
 
 namespace Volo.Abp.Http.Client;
 
-public class RemoteServiceConfigurationDictionary : Dictionary<string, RemoteServiceConfiguration>
+public class RemoteServiceConfigurationDictionary : Dictionary<string, RemoteServiceConfiguration?>
 {
     public const string DefaultName = "Default";
 
-    public RemoteServiceConfiguration Default {
+    public RemoteServiceConfiguration? Default {
         get => this.GetOrDefault(DefaultName);
         set => this[DefaultName] = value;
     }
@@ -20,8 +20,7 @@ public class RemoteServiceConfigurationDictionary : Dictionary<string, RemoteSer
                ?? throw new AbpException($"Remote service '{name}' was not found and there is no default configuration.");
     }
 
-    [CanBeNull]
+    public RemoteServiceConfiguration? GetConfigurationOrDefaultOrNull(string name)
-    public RemoteServiceConfiguration GetConfigurationOrDefaultOrNull(string name)
     {
         return this.GetOrDefault(name)
                ?? Default;

framework/src/Volo.Abp.RemoteServices/Volo/Abp/Http/Client/RemoteServiceConfigurationProvider.cs

@@ -18,7 +18,7 @@ public class RemoteServiceConfigurationProvider : IRemoteServiceConfigurationPro
         return Task.FromResult(Options.RemoteServices.GetConfigurationOrDefault(name));
     }
 
-    public Task<RemoteServiceConfiguration> GetConfigurationOrDefaultOrNullAsync(string name)
+    public Task<RemoteServiceConfiguration?> GetConfigurationOrDefaultOrNullAsync(string name)
     {
         return Task.FromResult(Options.RemoteServices.GetConfigurationOrDefaultOrNull(name));
     }

framework/src/Volo.Abp.RemoteServices/Volo/Abp/Http/Client/RemoteServiceConfigurationProviderExtensions.cs

@@ -10,8 +10,7 @@ public static class RemoteServiceConfigurationProviderExtensions
         this IRemoteServiceConfigurationProvider provider)
         => provider.GetConfigurationOrDefaultAsync(RemoteServiceConfigurationDictionary.DefaultName);
     
-    [ItemNotNull]
+    public static Task<RemoteServiceConfiguration?> GetConfigurationOrDefaultOrNullAsync(
-    public static Task<RemoteServiceConfiguration> GetConfigurationOrDefaultOrNullAsync(
         this IRemoteServiceConfigurationProvider provider)
         => provider.GetConfigurationOrDefaultOrNullAsync(RemoteServiceConfigurationDictionary.DefaultName);
 }