(GH-259) Actually fix todo comments

cake-contrib · Sep 30, 2019 · 0ee9b8f · 0ee9b8f
1 parent fd64332
commit 0ee9b8f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/Cake.Recipe/Content/azurepipelines.cake b/Cake.Recipe/Content/azurepipelines.cake
@@ -39,11 +39,7 @@ public class AzurePipelinesPullRequestInfo : IPullRequestInfo
 {
  public AzurePipelinesPullRequestInfo(ITFBuildProvider tfBuild, ICakeEnvironment environment)
  {
- var value = tfBuild.Environment.PullRequest.IsPullRequest;
-
- IsPullRequest = !string.IsNullOrWhiteSpace(value) && int.TryParse(value, out var id)
- ? id > 0
- : false;
+ IsPullRequest = tfBuild.Environment.PullRequest.IsPullRequest;
  }
 
  public bool IsPullRequest { get; }

diff --git a/Cake.Recipe/Content/buildProvider.cake b/Cake.Recipe/Content/buildProvider.cake
@@ -37,7 +37,7 @@ public interface IBuildProvider
 
 public static IBuildProvider GetBuildProvider(ICakeContext context, BuildSystem buildSystem)
 {
- if (IsRunningOnAzurePipelines || IsRunningOnAzurePipelinesHosted)
+ if (buildSystem.IsRunningOnAzurePipelines || buildSystem.IsRunningOnAzurePipelinesHosted)
  {
  return new AzurePipelinesBuildProvider(buildSystem.TFBuild, context.Environment);
  }