Skip to content
This repository has been archived by the owner on Aug 31, 2024. It is now read-only.

Search by email function #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

salsakran
Copy link

Simple couple liners to add that api function.

return this;
};

// Search users by email for `query`.
gh.user.search_email = function (query, callback, context) {
Copy link
Owner

Choose a reason for hiding this comment

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

First off, sorry for taking so long to get back to you.

I would be happy to merge this in if you rename this function to gh.user.searchByEmail.

Thanks!

Copy link
Author

Choose a reason for hiding this comment

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

will do.

I made a bunch of other changes I'll kick your way as well.

On Tue, Mar 22, 2011 at 5:31 PM, fitzgen
[email protected]
wrote:

         return this;
     };

  •    // Search users by email for query.
  •    gh.user.search_email = function (query, callback, context) {

First off, sorry for taking so long to get back to you.

I would be happy to merge this in if you rename this function to gh.user.searchByEmail.

Thanks!

Reply to this email directly or view it on GitHub:
https://github.com/fitzgen/github-api/pull/1/files#r11051

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants