refactor: attach allegro sdk and

Author: ppiwow-apay

global.json

@@ -2,5 +2,8 @@
     "sdk": {
         "version": "6.0.100",
         "rollForward": "latestFeature"
+    },
+    "msbuild-sdks": {
+        "Allegro.DotnetSdk": "1.2.2"
     }
 }

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore.Demo/Controllers/ErrorHandlingController.cs

@@ -1,8 +1,6 @@
-using System;
 using System.ComponentModel.DataAnnotations;
 using System.Net;
 using Allegro.Extensions.AspNetCore.ErrorHandling;
-using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Mvc;
 
 namespace Allegro.Extensions.AspNetCore.Demo.Controllers;
@@ -14,7 +12,9 @@ public class ErrorHandlingController : ControllerBase
     [HttpGet("default")]
     public IActionResult Default()
     {
+#pragma warning disable CA2201
         throw new Exception("Default error handling for not mapped exception");
+#pragma warning restore CA2201
     }
 
     [HttpGet("validationException")]

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore.Demo/Program.cs

@@ -1,6 +1,3 @@
-using Microsoft.AspNetCore.Hosting;
-using Microsoft.Extensions.Hosting;
-
 namespace Allegro.Extensions.AspNetCore.Demo
 {
     public static class Program

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore.Demo/Startup.cs

@@ -1,10 +1,6 @@
-using System;
 using Allegro.Extensions.AspNetCore.Demo.Controllers;
 using Allegro.Extensions.AspNetCore.ErrorHandling;
 using Allegro.Extensions.AspNetCore.Extensions;
-using Microsoft.AspNetCore.Builder;
-using Microsoft.AspNetCore.Hosting;
-using Microsoft.Extensions.DependencyInjection;
 
 namespace Allegro.Extensions.AspNetCore.Demo
 {

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore.Tests.Unit/ErrorHandling/ErrorHandlingMiddlewareTests.cs

@@ -1,11 +1,6 @@
-using System;
-using System.Collections.Generic;
-using System.Collections.Immutable;
 using System.ComponentModel.DataAnnotations;
-using System.IO;
-using System.Linq;
+using System.Diagnostics.CodeAnalysis;
 using System.Text.Json;
-using System.Threading.Tasks;
 using Allegro.Extensions.AspNetCore.ErrorHandling;
 using Allegro.Extensions.AspNetCore.ErrorHandling.Internals;
 using FluentAssertions;
@@ -14,6 +9,7 @@
 
 namespace Allegro.Extensions.AspNetCore.Tests.Unit.ErrorHandling;
 
+[SuppressMessage("Usage", "CA2201:Do not raise reserved exception types", Justification = "Test purposes")]
 public class ErrorHandlingMiddlewareTests
 {
     [Fact]

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore.Tests.Unit/SkipControllerFeatureProviderTests.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
 using System.Reflection;
 using Allegro.Extensions.AspNetCore.Attributes;
 using Allegro.Extensions.AspNetCore.Features;

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/Attributes/SkipOnProdAttribute.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace Allegro.Extensions.AspNetCore.Attributes
 {
     /// <summary>

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/ErrorHandlingConfigurationBuilder.cs

@@ -1,5 +1,3 @@
-using System;
-using System.Collections.Generic;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Mvc;
 

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/ErrorHandlingMiddlewareExtensions.cs

@@ -1,4 +1,3 @@
-using System;
 using Allegro.Extensions.AspNetCore.ErrorHandling.Internals;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.Extensions.DependencyInjection;

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/Internals/ErrorHandlingMiddleware.cs

@@ -1,9 +1,5 @@
-using System;
-using System.Collections.Generic;
 using System.ComponentModel.DataAnnotations;
-using System.Linq;
 using System.Text;
-using System.Threading.Tasks;
 using Microsoft.AspNetCore.Http;
 
 namespace Allegro.Extensions.AspNetCore.ErrorHandling.Internals;

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/Internals/ErrorResponse.cs

@@ -1,5 +1,3 @@
-using System.Collections.Generic;
-
 namespace Allegro.Extensions.AspNetCore.ErrorHandling.Internals;
 
 internal record ErrorResponse(

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/Internals/InvalidModelStateResponseFactory.cs

@@ -1,5 +1,3 @@
-using System;
-using System.Linq;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.Mvc.ModelBinding;

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/Models.cs

@@ -1,5 +1,3 @@
-using System.Collections.Generic;
-using System.Linq;
 using Allegro.Extensions.AspNetCore.ErrorHandling.Internals;
 
 // ReSharper disable ClassNeverInstantiated.Global

src/Allegro.Extensions.AspNetCore/Allegro.Extensions.AspNetCore/Features/SkipControllerFeatureProvider.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
 using System.Reflection;
 using Allegro.Extensions.AspNetCore.Attributes;
 using Microsoft.AspNetCore.Mvc.ApplicationParts;

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Commands/Command.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Commands;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Commands/ICommandDispatcher.cs

@@ -1,5 +1,3 @@
-using System.Threading.Tasks;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Commands;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Commands/ICommandHandler.cs

@@ -1,5 +1,3 @@
-using System.Threading.Tasks;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Commands;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Commands/ICommandValidator.cs

@@ -1,5 +1,3 @@
-using System.Threading.Tasks;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Commands;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/DecoratorAttribute.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace Allegro.Extensions.Cqrs.Abstractions;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Queries/IQueryDispatcher.cs

@@ -1,6 +1,3 @@
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Queries;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Queries/IQueryHandler.cs

@@ -1,6 +1,3 @@
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Queries;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Queries/IQueryValidator.cs

@@ -1,6 +1,3 @@
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Queries;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Abstractions/Queries/Query.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace Allegro.Extensions.Cqrs.Abstractions.Queries;
 
 /// <summary>

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Demo/Commands/BarCommand.cs

@@ -1,4 +1,3 @@
-using System.Threading.Tasks;
 using Allegro.Extensions.Cqrs.Abstractions;
 using Allegro.Extensions.Cqrs.Abstractions.Commands;
 using FluentValidation;
@@ -40,7 +39,9 @@ public BarCommandHandler(ILogger<BarCommandHandler> logger)
 
     public Task Handle(BarCommand command)
     {
+#pragma warning disable CA1848
         _logger.LogInformation("Handle Bar");
+#pragma warning restore CA1848
         return Task.CompletedTask;
     }
 }
@@ -59,8 +60,10 @@ public BarCommandHandlerDecorator(ICommandHandler<BarCommand> decorated, ILogger
 
     public async Task Handle(BarCommand command)
     {
+#pragma warning disable CA1848
         _logger.LogInformation("Before handle");
         await _decorated.Handle(command);
         _logger.LogInformation("After handle");
+#pragma warning restore CA1848
     }
 }

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs.Demo/Queries/BarQuery.cs

@@ -55,9 +55,11 @@ public BarQueryHandlerDecorator(IQueryHandler<BarQuery, BarData> decorated, ILog
 
     public async Task<BarData> Handle(BarQuery query, CancellationToken cancellationToken)
     {
+#pragma warning disable CA1848
         _logger.LogInformation("Before handle");
         var result = await _decorated.Handle(query, cancellationToken);
         _logger.LogInformation("After handle");
+#pragma warning restore CA1848
         return result;
     }
 }

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs/Commands/CommandDispatcher.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Linq;
-using System.Threading.Tasks;
 using Allegro.Extensions.Cqrs.Abstractions.Commands;
 using Microsoft.Extensions.DependencyInjection;
 

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs/Commands/StartupExtensions.cs

@@ -1,4 +1,3 @@
-using System.Collections.Generic;
 using System.Reflection;
 using Allegro.Extensions.Cqrs.Abstractions;
 using Allegro.Extensions.Cqrs.Abstractions.Commands;

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs/Queries/QueryDispatcher.cs

@@ -1,8 +1,4 @@
-using System;
-using System.Linq;
 using System.Reflection;
-using System.Threading;
-using System.Threading.Tasks;
 using Allegro.Extensions.Cqrs.Abstractions.Queries;
 using Microsoft.Extensions.DependencyInjection;
 

src/Allegro.Extensions.Cqrs/Allegro.Extensions.Cqrs/Queries/StartupExtensions.cs

@@ -1,4 +1,3 @@
-using System.Collections.Generic;
 using System.Reflection;
 using Allegro.Extensions.Cqrs.Abstractions;
 using Allegro.Extensions.Cqrs.Abstractions.Queries;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres.Tests.Integration/DapperPostgresBinaryCopyClient.cs

@@ -1,6 +1,3 @@
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 using Allegro.Extensions.Dapper.Abstractions;
 using Allegro.Extensions.Dapper.Postgres.Abstractions;
 using Allegro.Extensions.Dapper.Postgres.Tests.Integration.Helpers;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres.Tests.Integration/Helpers/CustomWebApplicationFactory.cs

@@ -1,4 +1,3 @@
-using System.IO;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Mvc.Testing;
 using Microsoft.AspNetCore.TestHost;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres.Tests.Integration/PostgresDapperClient.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 using Allegro.Extensions.Dapper.Abstractions;
 using Allegro.Extensions.Dapper.Postgres.Tests.Integration.Helpers;
 using FluentAssertions;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres/Abstractions/DbType.cs

@@ -1,9 +1,12 @@
+using System.Diagnostics.CodeAnalysis;
+
 #pragma warning disable CS1591
 namespace Allegro.Extensions.Dapper.Postgres.Abstractions;
 
 /// <summary>
 /// Dto database type.
 /// </summary>
+[SuppressMessage("Naming", "CA1720:Identifier contains type name", Justification = "Purpose of that enum")]
 public enum DbType
 {
     Int,

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres/Abstractions/IDapperPostgresBinaryCopyClient.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Threading.Tasks;
 using Allegro.Extensions.Dapper.Postgres.Exceptions;
 
 namespace Allegro.Extensions.Dapper.Postgres.Abstractions;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres/DapperPostgresBinaryCopyClient.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 using Allegro.Extensions.Dapper.Postgres.Abstractions;
 using Allegro.Extensions.Dapper.Postgres.Exceptions;
 using Allegro.Extensions.Dapper.Postgres.Factories;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres/Exceptions/InvalidDbConnectionTypeException.cs

@@ -1,4 +1,3 @@
-using System;
 #pragma warning disable CS1591
 
 namespace Allegro.Extensions.Dapper.Postgres.Exceptions;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper.Postgres/Extensions/ServiceCollectionExtensions.cs

@@ -1,4 +1,3 @@
-using System;
 using Allegro.Extensions.Dapper.Abstractions;
 using Allegro.Extensions.Dapper.Configurations;
 using Allegro.Extensions.Dapper.Postgres.Abstractions;

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper/Abstractions/IDapperClient.cs

@@ -1,6 +1,4 @@
-using System.Collections.Generic;
 using System.Data;
-using System.Threading.Tasks;
 
 namespace Allegro.Extensions.Dapper.Abstractions;
 

src/Allegro.Extensions.Dapper/Allegro.Extensions.Dapper/DapperClient.cs

@@ -1,7 +1,4 @@
-using System;
-using System.Collections.Generic;
 using System.Data;
-using System.Threading.Tasks;
 using Allegro.Extensions.Dapper.Abstractions;
 using Dapper;
 

src/Allegro.Extensions.Financials/Allegro.Extensions.Financials.Tests.Unit/Extensions/MoneyExtensionsTests.cs

@@ -1,4 +1,3 @@
-using System.Collections.Generic;
 using Allegro.Extensions.Financials.Extensions;
 using Allegro.Extensions.Financials.ValueObjects;
 using FluentAssertions;

src/Allegro.Extensions.Financials/Allegro.Extensions.Financials.Tests.Unit/ValueObjects/MoneyTests.cs

@@ -1,10 +1,12 @@
-using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using Allegro.Extensions.Financials.ValueObjects;
 using FluentAssertions;
 using Xunit;
 
 namespace Allegro.Extensions.Financials.Tests.Unit.ValueObjects;
 
+[SuppressMessage("Performance", "CA1825:Avoid zero-length array allocations", Justification = "Test class")]
+[SuppressMessage("Performance", "MA0005:Use Array.Empty<T>()", Justification = "Test class")]
 public class MoneyTests
 {
     public static IEnumerable<object?[]> DataValidAddMoney => new[]

src/Allegro.Extensions.Financials/Allegro.Extensions.Financials/Extensions/MoneyExtensions.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
 using Allegro.Extensions.Financials.ValueObjects;
 
 namespace Allegro.Extensions.Financials.Extensions;

src/Allegro.Extensions.Financials/Allegro.Extensions.Financials/ValueObjects/Money.Operators.cs

@@ -1,4 +1,3 @@
-using System;
 #pragma warning disable CS1591
 
 namespace Allegro.Extensions.Financials.ValueObjects;
@@ -100,9 +99,10 @@ private static Money Calculate(
 
         if (c1 != c2)
         {
-            throw new Exception(
-                $"Invalid currency! {c1} " +
-                $"is different than {c2}");
+#pragma warning disable CA2201
+            // TODO: more explicit exception should be added
+            throw new Exception($"Invalid currency! {c1} is different than {{c2}}");
+#pragma warning restore CA2201
         }
 
         var amount = @operator switch
@@ -131,8 +131,10 @@ private static bool Compare(
 
         if (c1 != c2)
         {
-            throw new Exception($"Invalid currency! {c1} " +
-                                $"is different than {c2}");
+#pragma warning disable CA2201
+            // TODO: more explicit exception should be added
+            throw new Exception($"Invalid currency! {c1} is different than {c2}");
+#pragma warning restore CA2201
         }
 
         return @operator switch

src/Allegro.Extensions.Globalization/Allegro.Extensions.Globalization.Tests/PolishPluralizerTests.cs

@@ -39,7 +39,9 @@ public class PolishPluralizerTests
         [InlineData(113, "spłat", "spłata", "spłaty", "spłat", "spłat")]
         [InlineData(124, "spłat", "spłata", "spłaty", "spłat", "spłaty")]
         [InlineData(127, "spłat", "spłata", "spłaty", "spłat", "spłat")]
+#pragma warning disable CA1720
         public void Pluralize_SampleText_RendersProperty(int quantity, string none, string single, string two, string five, string expected)
+#pragma warning restore CA1720
         {
             // act
             var result = quantity.Pluralize(