Merge pull request #10 from sentemon/fix-all-warnings

Fix warnings

backend/src/AuthService/AuthService.Domain/Entities/User.cs

@@ -13,8 +13,6 @@ public class User
     public string ImageUrl { get; private set; }
     public DateTime CreatedAt { get; private set; }
 
-    private User() { }
-
     private User(string id, string firstName, string lastName, Username username, Email email, string? imageUrl = null)
     {
         Id = id;
@@ -56,4 +54,9 @@ public void VerifyEmail()
     {
         EmailVerified = true;
     }
+
+    #pragma warning disable CS8618
+    // Required by EF Core
+    private User() { }
+    #pragma warning restore CS8618
 }

backend/src/Shared/Shared.Application/Abstractions/ICommand.cs

@@ -1,5 +1,3 @@
 namespace Shared.Application.Abstractions;
 
-public interface ICommand
-{
-}
+public interface ICommand;

backend/src/Shared/Shared.Application/Abstractions/IQuery.cs

@@ -1,5 +1,3 @@
 namespace Shared.Application.Abstractions;
 
-public interface IQuery
-{
-}
+public interface IQuery;

backend/src/Shared/Shared.Application/Common/Result.cs

@@ -6,21 +6,25 @@ namespace Shared.Application.Common;
 public class Result<TResponse> : IResult<TResponse, Error>
 {
 
+    #pragma warning disable CS8766 
     public TResponse? Response { get; }
     public Error? Error { get; }
+    #pragma warning restore CS8766
     public bool IsSuccess { get; }
     public HttpStatusCode StatusCode { get; }
 
-    private Result(TResponse? response)
+    private Result(TResponse response)
     {
         Response = response;
+        Error = default;
         IsSuccess = true;
         StatusCode = HttpStatusCode.OK;
     }
 
-    private Result(Error? error)
+    private Result(Error error)
     {
         Error = error;
+        Response = default;
         IsSuccess = false;
         StatusCode = HttpStatusCode.BadRequest;
     }