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

Docs: search users #952

Closed
wants to merge 5 commits into from
Closed

Conversation

hahmed
Copy link
Contributor

@hahmed hahmed commented Oct 30, 2015

Search dem users documentation.

Edit: something gone wrong with the commit history, sorry!

Naveen and others added 4 commits October 16, 2015 14:29
This commit  addressed the `BuildResponse`  wasn't handling
response `content-type` `application/octet-stream` for binary items.
Fixed the spacing of comma and aligned the arguments.
Fixes for Downloading ReleaseAsset zip File
@shiftkey
Copy link
Member

If you want to get rid of those changes:

git reset octokit/documentation --hard
git cherry-pick d80b8a3

and then force push it away...

@hahmed hahmed force-pushed the docs-search-users branch from d80b8a3 to 04c96e2 Compare October 30, 2015 22:46
@hahmed
Copy link
Contributor Author

hahmed commented Oct 30, 2015

@shiftkey force push it away? Sorry, I do not have a clue what that means.

git push origin docs-search-users --force ?

@shiftkey
Copy link
Member

@hahmed that's the one!

@haacked
Copy link
Contributor

haacked commented Oct 30, 2015

Be careful with --force. It rewrites history on the server. It's probably fine in this case, but if you ever do it to master on a shared project, you'll be making a lot of people unhappy. 😄

@hahmed
Copy link
Contributor Author

hahmed commented Oct 30, 2015

I just get an everything is up to date. Nothing has changed in this PR. Looks like I am doing something wrong. @haacked thanks for the heads up, I will keep that in mind.

@hahmed hahmed closed this Oct 30, 2015
@hahmed hahmed deleted the docs-search-users branch October 30, 2015 23:14
@hahmed hahmed mentioned this pull request Oct 30, 2015
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.

3 participants