Remove obsolete/deprecated MergeCommitSha property

Add Merged boolean property, returning MergedAt.HasValue
This commit is contained in:
Ryan Gribble
2015-12-13 23:35:43 +10:00
parent cb3a409d87
commit 2da609a724
+6 -4
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? 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, bool? mergeable, User mergedBy, int comments, int commits, int additions, int deletions, int changedFiles)
{
Url = url;
HtmlUrl = htmlUrl;
@@ -34,7 +34,6 @@ namespace Octokit
Base = @base;
User = user;
Assignee = assignee;
MergeCommitSha = mergeCommitSha;
Mergeable = mergeable;
MergedBy = mergedBy;
Comments = comments;
@@ -135,9 +134,12 @@ namespace Octokit
public User Assignee { get; protected set; }
/// <summary>
/// The SHA of the merge commit.
/// Whether or not the pull request has been merged.
/// </summary>
public string MergeCommitSha { get; protected set; }
public bool Merged
{
get { return MergedAt.HasValue; }
}
/// <summary>
/// Whether or not the pull request can be merged.