-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add new apps endpoint #932
Conversation
1 similar comment
spec/octokit/client/apps_spec.rb
Outdated
@@ -49,6 +49,14 @@ | |||
end | |||
end # .installation | |||
|
|||
describe ".find_installation_repositories_for_user", :vcr do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You shouldn't need to pass the :vcr
option here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the minor comment, this looks great! Requesting changes so that we're reminded to check back once the VCR cassettes are added.
Thanks.
1 similar comment
While working on VCR for octokit#932, I noticed that this other endpoint had a VCR entry that didn't correspond to the real response format. This fixes the test and updates the VCR cassette.
what do you think @kytrinyx ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this looks good. I've pointed out a ridiculously minor typo in the doc comment.
lib/octokit/client/apps.rb
Outdated
|
||
# List repositories accessible to the user for an installation | ||
# | ||
# @param installation [Integer] The id of a a GitHub App Installation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"a" is here twice.
@kytrinyx Fixed that another another grammatical error that were present in multiple places in this file - thanks! |
Adds
find_installation_repositories_for_user
Replacement for #904
Will add VCR once we're settled.