removed merged property for PR. also removed related tests for that property.

This commit is contained in:
Jiaming Chen
2015-10-10 11:26:10 -07:00
committed by Ryan Gribble
parent 1a74e9c7c3
commit cb3a409d87
2 changed files with 1 additions and 9 deletions
@@ -35,9 +35,7 @@ public class PullRequestsClientTests : IDisposable
var newPullRequest = new NewPullRequest("a pull request", branchName, "master");
var result = await _fixture.Create(Helper.UserName, _context.RepositoryName, newPullRequest);
Assert.Equal("a pull request", result.Title);
Assert.False(result.Merged);
}
[IntegrationTest]
+1 -7
View File
@@ -14,7 +14,7 @@ namespace Octokit
Number = number;
}
public PullRequest(Uri url, Uri htmlUrl, Uri diffUrl, Uri patchUrl, Uri issueUrl, Uri statusesUrl, int number, ItemState state, string title, string body, DateTimeOffset createdAt, DateTimeOffset updatedAt, DateTimeOffset? closedAt, DateTimeOffset? mergedAt, GitReference head, GitReference @base, User user, User assignee, string mergeCommitSha, bool merged, bool? mergeable, User mergedBy, int comments, int commits, int additions, int deletions, int changedFiles)
public PullRequest(Uri url, Uri htmlUrl, Uri diffUrl, Uri patchUrl, Uri issueUrl, Uri statusesUrl, int number, ItemState state, string title, string body, DateTimeOffset createdAt, DateTimeOffset updatedAt, DateTimeOffset? closedAt, DateTimeOffset? mergedAt, GitReference head, GitReference @base, User user, User assignee, string mergeCommitSha, bool? mergeable, User mergedBy, int comments, int commits, int additions, int deletions, int changedFiles)
{
Url = url;
HtmlUrl = htmlUrl;
@@ -35,7 +35,6 @@ namespace Octokit
User = user;
Assignee = assignee;
MergeCommitSha = mergeCommitSha;
Merged = merged || MergedAt.HasValue;
Mergeable = mergeable;
MergedBy = mergedBy;
Comments = comments;
@@ -140,11 +139,6 @@ namespace Octokit
/// </summary>
public string MergeCommitSha { get; protected set; }
/// <summary>
/// Whether or not the pull request has been merged.
/// </summary>
public bool Merged { get; protected set; }
/// <summary>
/// Whether or not the pull request can be merged.
/// </summary>