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

Implementation of Search Api #203

Closed
wants to merge 117 commits into from

Conversation

hahmed
Copy link
Contributor

@hahmed hahmed commented Nov 9, 2013

Not sure how to merge all of these into the pr #189
done something wrong here.

By the way - I have 3 failing tests.

Search Repo
Search Code
Search Issues

Note sure why?

http://developer.github.com/v3/search/
#137

jrsconfitto and others added 30 commits November 3, 2013 21:33
Conflicts:
	Octokit/Helpers/ApiUrls.cs
Change the `DeleteAsync` method to return `Task<IResponse<object>>` in
order to check the status code
* Add all `StarredClient` methods to the interface
* Add xml doc comments
* Correct parameter names
@hahmed
Copy link
Contributor Author

hahmed commented Nov 14, 2013

@shiftkey looks like this PR needs to be closed and open a new one that will get merged into master? I am merging my changes onto the incorrect branch?

How do you want me to do this? Thanks

@hahmed
Copy link
Contributor Author

hahmed commented Nov 14, 2013

@shiftkey what do you want me to do about: #203 (comment) ?

@hahmed
Copy link
Contributor Author

hahmed commented Nov 14, 2013

That rebase looks real ugly... Something went 💥

@shiftkey
Copy link
Member

@hahmed my psychic debugger thinks you need to get master up to date before doing the rebase

@hahmed hahmed mentioned this pull request Nov 14, 2013
10 tasks
@hahmed
Copy link
Contributor Author

hahmed commented Nov 14, 2013

replaced with #226

@hahmed hahmed closed this Nov 14, 2013
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.

7 participants