fix: remove random tap methods I added for no reason

Author: draekien

src/FluentUtils.Monad/OkResultType.cs

@@ -14,6 +14,7 @@ public sealed record OkResultType<T> : ResultType<T>
     internal OkResultType(T value, ILogger? logger = null)
     {
         Value = value;
+        if (logger is not null) Logger = logger;
     }
 
     internal T Value { get; }

src/FluentUtils.Monad/Operators/TapOperator.cs

@@ -123,60 +123,4 @@ public static async Task<ResultType<TIn>> Tap<TIn>(
         this Task<ResultType<TIn>> resultTask,
         Action<TIn> tap) =>
         (await resultTask).Tap(tap);
-
-    /// <summary>
-    ///     Tap into the value of a <see cref="ResultType{T}" /> to perform a
-    ///     side effect
-    /// </summary>
-    /// <remarks>
-    ///     The tap operator will only be executed if the input
-    ///     <see cref="ResultType{T}" /> is an <see cref="OkResultType{T}" />
-    /// </remarks>
-    /// <param name="resultTask">The <see cref="ResultType{T}" /></param>
-    /// <param name="tap">The side effect</param>
-    /// <typeparam name="TIn">The result value's type</typeparam>
-    /// <returns>The original result</returns>
-    [DebuggerStepperBoundary]
-    public static async Task<ResultType<TIn>> Tap<TIn>(
-        this Task<ResultType<Task<TIn>>> resultTask,
-        Action<TIn> tap)
-    {
-        ResultType<Task<TIn>> result = await resultTask;
-        return await result.Match(
-            async valueTask =>
-            {
-                TIn value = await valueTask;
-                tap(value);
-                return await Result.OkAsync(value, result.Logger);
-            },
-            error => Result.ErrorAsync<TIn>(error, result.Logger));
-    }
-
-    /// <summary>
-    ///     Tap into the value of a <see cref="ResultType{T}" /> to perform a
-    ///     side effect
-    /// </summary>
-    /// <remarks>
-    ///     The tap operator will only be executed if the input
-    ///     <see cref="ResultType{T}" /> is an <see cref="OkResultType{T}" />
-    /// </remarks>
-    /// <param name="resultTask">The <see cref="ResultType{T}" /></param>
-    /// <param name="tap">The side effect</param>
-    /// <typeparam name="TIn">The result value's type</typeparam>
-    /// <returns>The original result</returns>
-    [DebuggerStepperBoundary]
-    public static async Task<ResultType<TIn>> Tap<TIn>(
-        this Task<ResultType<Task<TIn>>> resultTask,
-        Func<TIn, Task> tap)
-    {
-        ResultType<Task<TIn>> result = await resultTask;
-        return await result.Match(
-            async valueTask =>
-            {
-                TIn value = await valueTask;
-                await tap(value);
-                return await Result.OkAsync(value, result.Logger);
-            },
-            error => Result.ErrorAsync<TIn>(error, result.Logger));
-    }
 }