Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Repository Deploy Keys client #512

Merged
merged 11 commits into from
Jul 3, 2014

Conversation

hnrkndrssn
Copy link
Contributor

@hnrkndrssn
Copy link
Contributor Author

As this is just waiting for me to do integration tests, the main changes are ready for review if someone has the time. 😁

@haacked
Copy link
Contributor

haacked commented Jul 1, 2014

Looks great! Any idea when you'll get to the integration tests?

@hnrkndrssn
Copy link
Contributor Author

I'll try and get them done in the next couple of days

@hnrkndrssn hnrkndrssn changed the title [WIP] Implement Repository Deploy Keys client Implement Repository Deploy Keys client Jul 2, 2014
@hnrkndrssn
Copy link
Contributor Author

Integration tests are now dun! 😄

// @haacked

@haacked
Copy link
Contributor

haacked commented Jul 2, 2014

That failure seems unrelated to your work. Thanks!

/// See the <a href="https://developer.github.com/v3/repos/keys/">Repository Deploy Keys API documentation</a> for more information.
/// </remarks>
IObservableRepositoryDeployKeysClient DeployKeys { get; }

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for this newline between the closing semi-colon and the code above.

@haacked
Copy link
Contributor

haacked commented Jul 2, 2014

@hnrkndrssn
Copy link
Contributor Author

@haacked anything else missing before this can be merged?

haacked added a commit that referenced this pull request Jul 3, 2014
Implement Repository Deploy Keys client
@haacked haacked merged commit be34685 into octokit:master Jul 3, 2014
@haacked
Copy link
Contributor

haacked commented Jul 3, 2014

Whoops, I thought I had merged it, but I got distracted. 😄

@hnrkndrssn
Copy link
Contributor Author

💖

@hnrkndrssn hnrkndrssn deleted the repo-deploykeys-client branch July 3, 2014 00:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Repository Deploy Keys API
2 participants