Merge pull request #657 from arturcic/feature/dotnet-format

gep13 · web-flow · commit 1fd47d12a320 · 2025-01-10T08:56:22.000-08:00
formatted the code
diff --git a/src/GitReleaseManager.Core.Tests/Provider/GitHubProviderTests.cs b/src/GitReleaseManager.Core.Tests/Provider/GitHubProviderTests.cs
@@ -20,7 +20,7 @@
 using NotFoundException = GitReleaseManager.Core.Exceptions.NotFoundException;
 using RateLimit = GitReleaseManager.Core.Model.RateLimit;
 using Release = GitReleaseManager.Core.Model.Release;
-using ReleaseAsset= GitReleaseManager.Core.Model.ReleaseAsset;
+using ReleaseAsset = GitReleaseManager.Core.Model.ReleaseAsset;
 using ReleaseAssetUpload = GitReleaseManager.Core.Model.ReleaseAssetUpload;
 
 namespace GitReleaseManager.Core.Tests.Provider
diff --git a/src/GitReleaseManager.Core.Tests/VcsServiceTests.cs b/src/GitReleaseManager.Core.Tests/VcsServiceTests.cs
@@ -350,8 +350,7 @@ public async Task Should_Create_Release_From_Milestone_With_Assets()
                     null,
                     _assets,
                     false,
-                    null
-                ).ConfigureAwait(false);
+                    null).ConfigureAwait(false);
             result.ShouldBeSameAs(release);
 
             await _releaseNotesBuilder.Received(1).BuildReleaseNotesAsync(OWNER, REPOSITORY, MILESTONE_TITLE, ReleaseTemplates.DEFAULT_NAME).ConfigureAwait(false);
diff --git a/src/GitReleaseManager.Core/Exceptions/InvalidIssuesException.cs b/src/GitReleaseManager.Core/Exceptions/InvalidIssuesException.cs
@@ -37,4 +37,4 @@ protected InvalidIssuesException(
         {
         }
     }
-}
+}
diff --git a/src/GitReleaseManager.Core/MappingProfiles/GitLabProfile.cs b/src/GitReleaseManager.Core/MappingProfiles/GitLabProfile.cs
@@ -45,4 +45,4 @@ public GitLabProfile()
                 .ReverseMap();
         }
     }
-}
+}
diff --git a/src/GitReleaseManager.Core/Model/VcsProvider.cs b/src/GitReleaseManager.Core/Model/VcsProvider.cs
@@ -5,4 +5,4 @@ public enum VcsProvider
         GitHub = 0,
         GitLab = 1,
     }
-}
+}
diff --git a/src/GitReleaseManager.Core/Options/BaseSubOptions.cs b/src/GitReleaseManager.Core/Options/BaseSubOptions.cs
@@ -9,8 +9,7 @@ protected BaseSubOptions()
         {
             var ciEnvironmentVariable = Environment.GetEnvironmentVariable("CI");
 
-            bool isCiSystem;
-            if (!string.IsNullOrEmpty(ciEnvironmentVariable) && bool.TryParse(ciEnvironmentVariable, out isCiSystem))
+            if (!string.IsNullOrEmpty(ciEnvironmentVariable) && bool.TryParse(ciEnvironmentVariable, out bool isCiSystem))
             {
                 IsCISystem = isCiSystem;
             }
diff --git a/src/GitReleaseManager.Core/Provider/GitHubProvider.cs b/src/GitReleaseManager.Core/Provider/GitHubProvider.cs
@@ -37,15 +37,15 @@ public GitHubProvider(IGitHubClient gitHubClient, IMapper mapper)
 
         public Task DeleteAssetAsync(string owner, string repository, ReleaseAsset asset)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 await _gitHubClient.Repository.Release.DeleteAsset(owner, repository, asset.Id).ConfigureAwait(false);
             });
         }
 
         public Task UploadAssetAsync(Release release, ReleaseAssetUpload releaseAssetUpload)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var octokitRelease = _mapper.Map<Octokit.Release>(release);
                 var octokitReleaseAssetUpload = _mapper.Map<Octokit.ReleaseAssetUpload>(releaseAssetUpload);
@@ -56,7 +56,7 @@ public Task UploadAssetAsync(Release release, ReleaseAssetUpload releaseAssetUpl
 
         public Task<int> GetCommitsCountAsync(string owner, string repository, string @base, string head)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 try
                 {
@@ -87,15 +87,15 @@ public string GetCommitsUrl(string owner, string repository, string head, string
 
         public Task CreateIssueCommentAsync(string owner, string repository, Issue issue, string comment)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 await _gitHubClient.Issue.Comment.Create(owner, repository, issue.PublicNumber, comment).ConfigureAwait(false);
             });
         }
 
         public Task<IEnumerable<Issue>> GetIssuesAsync(string owner, string repository, Milestone milestone, ItemStateFilter itemStateFilter = ItemStateFilter.All)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var openIssueRequest = new RepositoryIssueRequest
                 {
@@ -109,7 +109,7 @@ public Task<IEnumerable<Issue>> GetIssuesAsync(string owner, string repository,
 
                 do
                 {
-                    var options = GetApiOptions(startPage);
+                    var options = GitHubProvider.GetApiOptions(startPage);
                     results = await _gitHubClient.Issue.GetAllForRepository(owner, repository, openIssueRequest, options).ConfigureAwait(false);
 
                     issues.AddRange(results);
@@ -123,15 +123,15 @@ public Task<IEnumerable<Issue>> GetIssuesAsync(string owner, string repository,
 
         public Task<IEnumerable<IssueComment>> GetIssueCommentsAsync(string owner, string repository, Issue issue)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var startPage = 1;
                 var comments = new List<Octokit.IssueComment>();
                 IReadOnlyList<Octokit.IssueComment> results;
 
                 do
                 {
-                    var options = GetApiOptions(startPage);
+                    var options = GitHubProvider.GetApiOptions(startPage);
                     results = await _gitHubClient.Issue.Comment.GetAllForIssue(owner, repository, issue.PublicNumber, options).ConfigureAwait(false);
 
                     comments.AddRange(results);
@@ -145,7 +145,7 @@ public Task<IEnumerable<IssueComment>> GetIssueCommentsAsync(string owner, strin
 
         public Task CreateLabelAsync(string owner, string repository, Label label)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var newLabel = _mapper.Map<NewLabel>(label);
 
@@ -155,23 +155,23 @@ public Task CreateLabelAsync(string owner, string repository, Label label)
 
         public Task DeleteLabelAsync(string owner, string repository, Label label)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 await _gitHubClient.Issue.Labels.Delete(owner, repository, label.Name).ConfigureAwait(false);
             });
         }
 
         public Task<IEnumerable<Label>> GetLabelsAsync(string owner, string repository)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var startPage = 1;
                 var labels = new List<Octokit.Label>();
                 IReadOnlyList<Octokit.Label> results;
 
                 do
                 {
-                    var options = GetApiOptions(startPage);
+                    var options = GitHubProvider.GetApiOptions(startPage);
                     results = await _gitHubClient.Issue.Labels.GetAllForRepository(owner, repository, options).ConfigureAwait(false);
 
                     labels.AddRange(results);
@@ -185,7 +185,7 @@ public Task<IEnumerable<Label>> GetLabelsAsync(string owner, string repository)
 
         public Task<Milestone> GetMilestoneAsync(string owner, string repository, string milestoneTitle, ItemStateFilter itemStateFilter = ItemStateFilter.All)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var milestones = await GetMilestonesAsync(owner, repository, itemStateFilter).ConfigureAwait(false);
                 var foundMilestone = milestones.FirstOrDefault(m => m.Title == milestoneTitle);
@@ -201,7 +201,7 @@ public Task<Milestone> GetMilestoneAsync(string owner, string repository, string
 
         public Task<IEnumerable<Milestone>> GetMilestonesAsync(string owner, string repository, ItemStateFilter itemStateFilter = ItemStateFilter.All)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var request = new MilestoneRequest { State = (Octokit.ItemStateFilter)itemStateFilter };
 
@@ -211,7 +211,7 @@ public Task<IEnumerable<Milestone>> GetMilestonesAsync(string owner, string repo
 
                 do
                 {
-                    var options = GetApiOptions(startPage);
+                    var options = GitHubProvider.GetApiOptions(startPage);
                     results = await _gitHubClient.Issue.Milestone.GetAllForRepository(owner, repository, request, options).ConfigureAwait(false);
 
                     milestones.AddRange(results);
@@ -225,7 +225,7 @@ public Task<IEnumerable<Milestone>> GetMilestonesAsync(string owner, string repo
 
         public Task SetMilestoneStateAsync(string owner, string repository, Milestone milestone, ItemState itemState)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var update = new MilestoneUpdate { State = (Octokit.ItemState)itemState };
                 await _gitHubClient.Issue.Milestone.Update(owner, repository, milestone.PublicNumber, update).ConfigureAwait(false);
@@ -234,7 +234,7 @@ public Task SetMilestoneStateAsync(string owner, string repository, Milestone mi
 
         public Task<Release> CreateReleaseAsync(string owner, string repository, Release release)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var newRelease = _mapper.Map<NewRelease>(release);
                 var octokitRelease = await _gitHubClient.Repository.Release.Create(owner, repository, newRelease).ConfigureAwait(false);
@@ -245,15 +245,15 @@ public Task<Release> CreateReleaseAsync(string owner, string repository, Release
 
         public Task DeleteReleaseAsync(string owner, string repository, Release release)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 await _gitHubClient.Repository.Release.Delete(owner, repository, release.Id).ConfigureAwait(false);
             });
         }
 
         public Task<Release> GetReleaseAsync(string owner, string repository, string tagName)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 // This method wants to return a single Release, that has the tagName that is requested.
                 // The obvious thing to do here would be to use Repository.Release.Get, however, this doesn't
@@ -270,15 +270,15 @@ public Task<Release> GetReleaseAsync(string owner, string repository, string tag
 
         public Task<IEnumerable<Release>> GetReleasesAsync(string owner, string repository, bool skipPrereleases)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var startPage = 1;
                 var releases = new List<Octokit.Release>();
                 IReadOnlyList<Octokit.Release> results;
 
                 do
                 {
-                    var options = GetApiOptions(startPage);
+                    var options = GitHubProvider.GetApiOptions(startPage);
                     results = await _gitHubClient.Repository.Release.GetAll(owner, repository, options).ConfigureAwait(false);
 
                     if (skipPrereleases)
@@ -302,7 +302,7 @@ public Task<IEnumerable<Release>> GetReleasesAsync(string owner, string reposito
 
         public Task PublishReleaseAsync(string owner, string repository, string tagName, Release release)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var update = new ReleaseUpdate
                 {
@@ -316,7 +316,7 @@ public Task PublishReleaseAsync(string owner, string repository, string tagName,
 
         public Task UpdateReleaseAsync(string owner, string repository, Release release)
         {
-            return ExecuteAsync(async () =>
+            return GitHubProvider.ExecuteAsync(async () =>
             {
                 var update = new ReleaseUpdate
                 {
@@ -356,7 +356,7 @@ public string GetIssueType(Issue issue)
             return issue.IsPullRequest ? "Pull Request" : "Issue";
         }
 
-        private async Task ExecuteAsync(Func<Task> action)
+        private static async Task ExecuteAsync(Func<Task> action)
         {
             try
             {
@@ -376,7 +376,7 @@ private async Task ExecuteAsync(Func<Task> action)
             }
         }
 
-        private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)
+        private static async Task<T> ExecuteAsync<T>(Func<Task<T>> action)
         {
             try
             {
@@ -396,7 +396,7 @@ private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)
             }
         }
 
-        private ApiOptions GetApiOptions(int startPage = 1, int pageSize = 100, int pageCount = 1)
+        private static ApiOptions GetApiOptions(int startPage = 1, int pageSize = 100, int pageCount = 1)
         {
             return new ApiOptions
             {
diff --git a/src/GitReleaseManager.Core/Provider/GitLabProvider.cs b/src/GitReleaseManager.Core/Provider/GitLabProvider.cs
@@ -405,7 +405,7 @@ private long GetGitLabProjectId(string owner, string repository)
             return _projectId.Value;
         }
 
-        private async Task ExecuteAsync(Func<Task> action)
+        private static async Task ExecuteAsync(Func<Task> action)
         {
             try
             {
@@ -421,7 +421,7 @@ private async Task ExecuteAsync(Func<Task> action)
             }
         }
 
-        private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)
+        private static async Task<T> ExecuteAsync<T>(Func<Task<T>> action)
         {
             try
             {
@@ -437,4 +437,4 @@ private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)
             }
         }
     }
-}
+}
diff --git a/src/GitReleaseManager.Core/ReleaseNotes/ReleaseNotesBuilder.cs b/src/GitReleaseManager.Core/ReleaseNotes/ReleaseNotesBuilder.cs
@@ -143,9 +143,9 @@ private string GetValidLabel(string label, int issuesCount)
 
                 foreach (var issueLabel in issue.Labels)
                 {
-                    includedIssuesCount += _configuration.IssueLabelsInclude.Count(issueToInclude => issueLabel.Name.ToUpperInvariant() == issueToInclude.ToUpperInvariant());
+                    includedIssuesCount += _configuration.IssueLabelsInclude.Count(issueToInclude => string.Equals(issueLabel.Name, issueToInclude, StringComparison.InvariantCultureIgnoreCase));
 
-                    isExcluded = isExcluded || _configuration.IssueLabelsExclude.Any(issueToExclude => issueLabel.Name.ToUpperInvariant() == issueToExclude.ToUpperInvariant());
+                    isExcluded = isExcluded || _configuration.IssueLabelsExclude.Any(issueToExclude => string.Equals(issueLabel.Name, issueToExclude, StringComparison.InvariantCultureIgnoreCase));
                 }
 
                 if (isExcluded)
diff --git a/src/GitReleaseManager.IntegrationTests/GitLabProviderIntegrationTests.cs b/src/GitReleaseManager.IntegrationTests/GitLabProviderIntegrationTests.cs
@@ -108,4 +108,4 @@ public async Task Should_Get_Commits_Count()
             // result.ShouldBeGreaterThan(0);
         }
     }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -37,4 +37,4 @@ protected InvalidIssuesException(`
`37`	`37`	`{`
`38`	`38`	`}`
`39`	`39`	`}`
`40`		`-}`
	`40`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -45,4 +45,4 @@ public GitLabProfile()`
`45`	`45`	`.ReverseMap();`
`46`	`46`	`}`
`47`	`47`	`}`
`48`		`-}`
	`48`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -5,4 +5,4 @@ public enum VcsProvider`
`5`	`5`	`GitHub = 0,`
`6`	`6`	`GitLab = 1,`
`7`	`7`	`}`
`8`		`-}`
	`8`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -9,8 +9,7 @@ protected BaseSubOptions()`
`9`	`9`	`{`
`10`	`10`	`var ciEnvironmentVariable = Environment.GetEnvironmentVariable("CI");`
`11`	`11`
`12`		`- bool isCiSystem;`
`13`		`- if (!string.IsNullOrEmpty(ciEnvironmentVariable) && bool.TryParse(ciEnvironmentVariable, out isCiSystem))`
	`12`	`+ if (!string.IsNullOrEmpty(ciEnvironmentVariable) && bool.TryParse(ciEnvironmentVariable, out bool isCiSystem))`
`14`	`13`	`{`
`15`	`14`	`IsCISystem = isCiSystem;`
`16`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,15 +37,15 @@ public GitHubProvider(IGitHubClient gitHubClient, IMapper mapper)`
`37`	`37`
`38`	`38`	`public Task DeleteAssetAsync(string owner, string repository, ReleaseAsset asset)`
`39`	`39`	`{`
`40`		`- return ExecuteAsync(async () =>`
	`40`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`41`	`41`	`{`
`42`	`42`	`await _gitHubClient.Repository.Release.DeleteAsset(owner, repository, asset.Id).ConfigureAwait(false);`
`43`	`43`	`});`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`public Task UploadAssetAsync(Release release, ReleaseAssetUpload releaseAssetUpload)`
`47`	`47`	`{`
`48`		`- return ExecuteAsync(async () =>`
	`48`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`49`	`49`	`{`
`50`	`50`	`var octokitRelease = _mapper.Map<Octokit.Release>(release);`
`51`	`51`	`var octokitReleaseAssetUpload = _mapper.Map<Octokit.ReleaseAssetUpload>(releaseAssetUpload);`
`@@ -56,7 +56,7 @@ public Task UploadAssetAsync(Release release, ReleaseAssetUpload releaseAssetUpl`
`56`	`56`
`57`	`57`	`public Task<int> GetCommitsCountAsync(string owner, string repository, string @base, string head)`
`58`	`58`	`{`
`59`		`- return ExecuteAsync(async () =>`
	`59`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`60`	`60`	`{`
`61`	`61`	`try`
`62`	`62`	`{`
`@@ -87,15 +87,15 @@ public string GetCommitsUrl(string owner, string repository, string head, string`
`87`	`87`
`88`	`88`	`public Task CreateIssueCommentAsync(string owner, string repository, Issue issue, string comment)`
`89`	`89`	`{`
`90`		`- return ExecuteAsync(async () =>`
	`90`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`91`	`91`	`{`
`92`	`92`	`await _gitHubClient.Issue.Comment.Create(owner, repository, issue.PublicNumber, comment).ConfigureAwait(false);`
`93`	`93`	`});`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`public Task<IEnumerable<Issue>> GetIssuesAsync(string owner, string repository, Milestone milestone, ItemStateFilter itemStateFilter = ItemStateFilter.All)`
`97`	`97`	`{`
`98`		`- return ExecuteAsync(async () =>`
	`98`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`99`	`99`	`{`
`100`	`100`	`var openIssueRequest = new RepositoryIssueRequest`
`101`	`101`	`{`
`@@ -109,7 +109,7 @@ public Task<IEnumerable<Issue>> GetIssuesAsync(string owner, string repository,`
`109`	`109`
`110`	`110`	`do`
`111`	`111`	`{`
`112`		`- var options = GetApiOptions(startPage);`
	`112`	`+ var options = GitHubProvider.GetApiOptions(startPage);`
`113`	`113`	`results = await _gitHubClient.Issue.GetAllForRepository(owner, repository, openIssueRequest, options).ConfigureAwait(false);`
`114`	`114`
`115`	`115`	`issues.AddRange(results);`
`@@ -123,15 +123,15 @@ public Task<IEnumerable<Issue>> GetIssuesAsync(string owner, string repository,`
`123`	`123`
`124`	`124`	`public Task<IEnumerable<IssueComment>> GetIssueCommentsAsync(string owner, string repository, Issue issue)`
`125`	`125`	`{`
`126`		`- return ExecuteAsync(async () =>`
	`126`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`127`	`127`	`{`
`128`	`128`	`var startPage = 1;`
`129`	`129`	`var comments = new List<Octokit.IssueComment>();`
`130`	`130`	`IReadOnlyList<Octokit.IssueComment> results;`
`131`	`131`
`132`	`132`	`do`
`133`	`133`	`{`
`134`		`- var options = GetApiOptions(startPage);`
	`134`	`+ var options = GitHubProvider.GetApiOptions(startPage);`
`135`	`135`	`results = await _gitHubClient.Issue.Comment.GetAllForIssue(owner, repository, issue.PublicNumber, options).ConfigureAwait(false);`
`136`	`136`
`137`	`137`	`comments.AddRange(results);`
`@@ -145,7 +145,7 @@ public Task<IEnumerable<IssueComment>> GetIssueCommentsAsync(string owner, strin`
`145`	`145`
`146`	`146`	`public Task CreateLabelAsync(string owner, string repository, Label label)`
`147`	`147`	`{`
`148`		`- return ExecuteAsync(async () =>`
	`148`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`149`	`149`	`{`
`150`	`150`	`var newLabel = _mapper.Map<NewLabel>(label);`
`151`	`151`
`@@ -155,23 +155,23 @@ public Task CreateLabelAsync(string owner, string repository, Label label)`
`155`	`155`
`156`	`156`	`public Task DeleteLabelAsync(string owner, string repository, Label label)`
`157`	`157`	`{`
`158`		`- return ExecuteAsync(async () =>`
	`158`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`159`	`159`	`{`
`160`	`160`	`await _gitHubClient.Issue.Labels.Delete(owner, repository, label.Name).ConfigureAwait(false);`
`161`	`161`	`});`
`162`	`162`	`}`
`163`	`163`
`164`	`164`	`public Task<IEnumerable<Label>> GetLabelsAsync(string owner, string repository)`
`165`	`165`	`{`
`166`		`- return ExecuteAsync(async () =>`
	`166`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`167`	`167`	`{`
`168`	`168`	`var startPage = 1;`
`169`	`169`	`var labels = new List<Octokit.Label>();`
`170`	`170`	`IReadOnlyList<Octokit.Label> results;`
`171`	`171`
`172`	`172`	`do`
`173`	`173`	`{`
`174`		`- var options = GetApiOptions(startPage);`
	`174`	`+ var options = GitHubProvider.GetApiOptions(startPage);`
`175`	`175`	`results = await _gitHubClient.Issue.Labels.GetAllForRepository(owner, repository, options).ConfigureAwait(false);`
`176`	`176`
`177`	`177`	`labels.AddRange(results);`
`@@ -185,7 +185,7 @@ public Task<IEnumerable<Label>> GetLabelsAsync(string owner, string repository)`
`185`	`185`
`186`	`186`	`public Task<Milestone> GetMilestoneAsync(string owner, string repository, string milestoneTitle, ItemStateFilter itemStateFilter = ItemStateFilter.All)`
`187`	`187`	`{`
`188`		`- return ExecuteAsync(async () =>`
	`188`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`189`	`189`	`{`
`190`	`190`	`var milestones = await GetMilestonesAsync(owner, repository, itemStateFilter).ConfigureAwait(false);`
`191`	`191`	`var foundMilestone = milestones.FirstOrDefault(m => m.Title == milestoneTitle);`
`@@ -201,7 +201,7 @@ public Task<Milestone> GetMilestoneAsync(string owner, string repository, string`
`201`	`201`
`202`	`202`	`public Task<IEnumerable<Milestone>> GetMilestonesAsync(string owner, string repository, ItemStateFilter itemStateFilter = ItemStateFilter.All)`
`203`	`203`	`{`
`204`		`- return ExecuteAsync(async () =>`
	`204`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`205`	`205`	`{`
`206`	`206`	`var request = new MilestoneRequest { State = (Octokit.ItemStateFilter)itemStateFilter };`
`207`	`207`
`@@ -211,7 +211,7 @@ public Task<IEnumerable<Milestone>> GetMilestonesAsync(string owner, string repo`
`211`	`211`
`212`	`212`	`do`
`213`	`213`	`{`
`214`		`- var options = GetApiOptions(startPage);`
	`214`	`+ var options = GitHubProvider.GetApiOptions(startPage);`
`215`	`215`	`results = await _gitHubClient.Issue.Milestone.GetAllForRepository(owner, repository, request, options).ConfigureAwait(false);`
`216`	`216`
`217`	`217`	`milestones.AddRange(results);`
`@@ -225,7 +225,7 @@ public Task<IEnumerable<Milestone>> GetMilestonesAsync(string owner, string repo`
`225`	`225`
`226`	`226`	`public Task SetMilestoneStateAsync(string owner, string repository, Milestone milestone, ItemState itemState)`
`227`	`227`	`{`
`228`		`- return ExecuteAsync(async () =>`
	`228`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`229`	`229`	`{`
`230`	`230`	`var update = new MilestoneUpdate { State = (Octokit.ItemState)itemState };`
`231`	`231`	`await _gitHubClient.Issue.Milestone.Update(owner, repository, milestone.PublicNumber, update).ConfigureAwait(false);`
`@@ -234,7 +234,7 @@ public Task SetMilestoneStateAsync(string owner, string repository, Milestone mi`
`234`	`234`
`235`	`235`	`public Task<Release> CreateReleaseAsync(string owner, string repository, Release release)`
`236`	`236`	`{`
`237`		`- return ExecuteAsync(async () =>`
	`237`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`238`	`238`	`{`
`239`	`239`	`var newRelease = _mapper.Map<NewRelease>(release);`
`240`	`240`	`var octokitRelease = await _gitHubClient.Repository.Release.Create(owner, repository, newRelease).ConfigureAwait(false);`
`@@ -245,15 +245,15 @@ public Task<Release> CreateReleaseAsync(string owner, string repository, Release`
`245`	`245`
`246`	`246`	`public Task DeleteReleaseAsync(string owner, string repository, Release release)`
`247`	`247`	`{`
`248`		`- return ExecuteAsync(async () =>`
	`248`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`249`	`249`	`{`
`250`	`250`	`await _gitHubClient.Repository.Release.Delete(owner, repository, release.Id).ConfigureAwait(false);`
`251`	`251`	`});`
`252`	`252`	`}`
`253`	`253`
`254`	`254`	`public Task<Release> GetReleaseAsync(string owner, string repository, string tagName)`
`255`	`255`	`{`
`256`		`- return ExecuteAsync(async () =>`
	`256`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`257`	`257`	`{`
`258`	`258`	`// This method wants to return a single Release, that has the tagName that is requested.`
`259`	`259`	`// The obvious thing to do here would be to use Repository.Release.Get, however, this doesn't`
`@@ -270,15 +270,15 @@ public Task<Release> GetReleaseAsync(string owner, string repository, string tag`
`270`	`270`
`271`	`271`	`public Task<IEnumerable<Release>> GetReleasesAsync(string owner, string repository, bool skipPrereleases)`
`272`	`272`	`{`
`273`		`- return ExecuteAsync(async () =>`
	`273`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`274`	`274`	`{`
`275`	`275`	`var startPage = 1;`
`276`	`276`	`var releases = new List<Octokit.Release>();`
`277`	`277`	`IReadOnlyList<Octokit.Release> results;`
`278`	`278`
`279`	`279`	`do`
`280`	`280`	`{`
`281`		`- var options = GetApiOptions(startPage);`
	`281`	`+ var options = GitHubProvider.GetApiOptions(startPage);`
`282`	`282`	`results = await _gitHubClient.Repository.Release.GetAll(owner, repository, options).ConfigureAwait(false);`
`283`	`283`
`284`	`284`	`if (skipPrereleases)`
`@@ -302,7 +302,7 @@ public Task<IEnumerable<Release>> GetReleasesAsync(string owner, string reposito`
`302`	`302`
`303`	`303`	`public Task PublishReleaseAsync(string owner, string repository, string tagName, Release release)`
`304`	`304`	`{`
`305`		`- return ExecuteAsync(async () =>`
	`305`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`306`	`306`	`{`
`307`	`307`	`var update = new ReleaseUpdate`
`308`	`308`	`{`
`@@ -316,7 +316,7 @@ public Task PublishReleaseAsync(string owner, string repository, string tagName,`
`316`	`316`
`317`	`317`	`public Task UpdateReleaseAsync(string owner, string repository, Release release)`
`318`	`318`	`{`
`319`		`- return ExecuteAsync(async () =>`
	`319`	`+ return GitHubProvider.ExecuteAsync(async () =>`
`320`	`320`	`{`
`321`	`321`	`var update = new ReleaseUpdate`
`322`	`322`	`{`
`@@ -356,7 +356,7 @@ public string GetIssueType(Issue issue)`
`356`	`356`	`return issue.IsPullRequest ? "Pull Request" : "Issue";`
`357`	`357`	`}`
`358`	`358`
`359`		`- private async Task ExecuteAsync(Func<Task> action)`
	`359`	`+ private static async Task ExecuteAsync(Func<Task> action)`
`360`	`360`	`{`
`361`	`361`	`try`
`362`	`362`	`{`
`@@ -376,7 +376,7 @@ private async Task ExecuteAsync(Func<Task> action)`
`376`	`376`	`}`
`377`	`377`	`}`
`378`	`378`
`379`		`- private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)`
	`379`	`+ private static async Task<T> ExecuteAsync<T>(Func<Task<T>> action)`
`380`	`380`	`{`
`381`	`381`	`try`
`382`	`382`	`{`
`@@ -396,7 +396,7 @@ private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)`
`396`	`396`	`}`
`397`	`397`	`}`
`398`	`398`
`399`		`- private ApiOptions GetApiOptions(int startPage = 1, int pageSize = 100, int pageCount = 1)`
	`399`	`+ private static ApiOptions GetApiOptions(int startPage = 1, int pageSize = 100, int pageCount = 1)`
`400`	`400`	`{`
`401`	`401`	`return new ApiOptions`
`402`	`402`	`{`
Original file line number	Diff line number	Diff line change
`@@ -405,7 +405,7 @@ private long GetGitLabProjectId(string owner, string repository)`
`405`	`405`	`return _projectId.Value;`
`406`	`406`	`}`
`407`	`407`
`408`		`- private async Task ExecuteAsync(Func<Task> action)`
	`408`	`+ private static async Task ExecuteAsync(Func<Task> action)`
`409`	`409`	`{`
`410`	`410`	`try`
`411`	`411`	`{`
`@@ -421,7 +421,7 @@ private async Task ExecuteAsync(Func<Task> action)`
`421`	`421`	`}`
`422`	`422`	`}`
`423`	`423`
`424`		`- private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)`
	`424`	`+ private static async Task<T> ExecuteAsync<T>(Func<Task<T>> action)`
`425`	`425`	`{`
`426`	`426`	`try`
`427`	`427`	`{`
`@@ -437,4 +437,4 @@ private async Task<T> ExecuteAsync<T>(Func<Task<T>> action)`
`437`	`437`	`}`
`438`	`438`	`}`
`439`	`439`	`}`
`440`		`-}`
	`440`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -143,9 +143,9 @@ private string GetValidLabel(string label, int issuesCount)`
`143`	`143`
`144`	`144`	`foreach (var issueLabel in issue.Labels)`
`145`	`145`	`{`
`146`		`- includedIssuesCount += _configuration.IssueLabelsInclude.Count(issueToInclude => issueLabel.Name.ToUpperInvariant() == issueToInclude.ToUpperInvariant());`
	`146`	`+ includedIssuesCount += _configuration.IssueLabelsInclude.Count(issueToInclude => string.Equals(issueLabel.Name, issueToInclude, StringComparison.InvariantCultureIgnoreCase));`
`147`	`147`
`148`		`- isExcluded = isExcluded \|\| _configuration.IssueLabelsExclude.Any(issueToExclude => issueLabel.Name.ToUpperInvariant() == issueToExclude.ToUpperInvariant());`
	`148`	`+ isExcluded = isExcluded \|\| _configuration.IssueLabelsExclude.Any(issueToExclude => string.Equals(issueLabel.Name, issueToExclude, StringComparison.InvariantCultureIgnoreCase));`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`if (isExcluded)`
Original file line number	Diff line number	Diff line change
`@@ -108,4 +108,4 @@ public async Task Should_Get_Commits_Count()`
`108`	`108`	`// result.ShouldBeGreaterThan(0);`
`109`	`109`	`}`
`110`	`110`	`}`
`111`		`-}`
	`111`	`+}`