Remove DupFinder exclusions (#286)

cake-contrib · Aug 24, 2022 · 9f3eb99 · 9f3eb99
1 parent 46d9e8f
commit 9f3eb99
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 12 deletions.
diff --git a/...ues.Recipe/Cake.Frosting.Issues.Recipe/PullRequestSystems/AzureDevOpsPullRequestSystem.cs b/...ues.Recipe/Cake.Frosting.Issues.Recipe/PullRequestSystems/AzureDevOpsPullRequestSystem.cs
@@ -20,9 +20,6 @@ public override void ReportIssuesToPullRequest(IIssuesContext context)
         {
             context.NotNull(nameof(context));
 
-#pragma warning disable SA1123 // Do not place regions within elements
-            #region DupFinder Exclusion
-#pragma warning restore SA1123 // Do not place regions within elements
             if (string.IsNullOrWhiteSpace(context.EnvironmentVariable("SYSTEM_ACCESSTOKEN")))
             {
                 context.Warning("SYSTEM_ACCESSTOKEN environment variable not set. Make sure the 'Allow Scripts to access OAuth token' option is enabled on the build definition.");
@@ -36,7 +33,6 @@ public override void ReportIssuesToPullRequest(IIssuesContext context)
                     context.State.BuildServer.DeterminePullRequestId(context).Value,
                     context.AzureDevOpsAuthenticationOAuth(context.EnvironmentVariable("SYSTEM_ACCESSTOKEN"))),
                 GetReportIssuesToPullRequestSettings(context));
-            #endregion
         }
 
         /// <inheritdoc />

diff --git a/Cake.Frosting.Issues.Recipe/Cake.Frosting.Issues.Recipe/ReportIssuesToPullRequestTask.cs b/Cake.Frosting.Issues.Recipe/Cake.Frosting.Issues.Recipe/ReportIssuesToPullRequestTask.cs
@@ -11,9 +11,6 @@ namespace Cake.Frosting.Issues.Recipe
     [IsDependentOn(typeof(ReadIssuesTask))]
     public sealed class ReportIssuesToPullRequestTask : FrostingTask<IIssuesContext>
     {
-#pragma warning disable SA1123 // Do not place regions within elements
-        #region DupFinder Exclusion
-#pragma warning restore SA1123 // Do not place regions within elements
         /// <inheritdoc/>
         public override bool ShouldRun(IIssuesContext context)
         {
@@ -24,7 +21,6 @@ public override bool ShouldRun(IIssuesContext context)
                 context.Parameters.PullRequestSystem.ShouldReportIssuesToPullRequest &&
                 context.State.BuildServer != null && context.State.BuildServer.DetermineIfPullRequest(context);
         }
-        #endregion
 
         /// <inheritdoc/>
         public override void Run(IIssuesContext context)

diff --git a/Cake.Frosting.Issues.Recipe/Cake.Frosting.Issues.Recipe/SetPullRequestIssuesStateTask.cs b/Cake.Frosting.Issues.Recipe/Cake.Frosting.Issues.Recipe/SetPullRequestIssuesStateTask.cs
@@ -11,9 +11,6 @@ namespace Cake.Frosting.Issues.Recipe
     [IsDependentOn(typeof(ReadIssuesTask))]
     public sealed class SetPullRequestIssuesStateTask : FrostingTask<IIssuesContext>
     {
-#pragma warning disable SA1123 // Do not place regions within elements
-        #region DupFinder Exclusion
-#pragma warning restore SA1123 // Do not place regions within elements
         /// <inheritdoc/>
         public override bool ShouldRun(IIssuesContext context)
         {
@@ -24,7 +21,6 @@ public override bool ShouldRun(IIssuesContext context)
                 context.Parameters.PullRequestSystem.ShouldSetPullRequestStatus &&
                 context.State.BuildServer != null && context.State.BuildServer.DetermineIfPullRequest(context);
         }
-        #endregion
 
         /// <inheritdoc/>
         public override void Run(IIssuesContext context)