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: Return super in example Transport subclass #980

Merged
merged 3 commits into from
Dec 19, 2019

Conversation

nathanmoon
Copy link
Contributor

If called without a callback, the request method returns a Promise, so
when calling into super.request, the result should be returned to
maintain promise behavior.

If called without a callback, the request method returns a Promise, so
when calling into super.request, the result should be returned to
maintain promise behavior.
@nathanmoon nathanmoon changed the title Return super in example Transport subclass Docs: Return super in example Transport subclass Oct 4, 2019
@elasticcla
Copy link

Hi @nathanmoon, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in your Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

@nathanmoon
Copy link
Contributor Author

I added my email. Do I need to do something to trigger the check to run again?

@delvedor
Copy link
Member

jenkins run the tests please

Copy link
Member

@delvedor delvedor left a comment

Choose a reason for hiding this comment

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

Thanks!

@stale
Copy link

stale bot commented Oct 25, 2019

We understand that this might be important for you, but this issue has been automatically marked as stale because it has not had recent activity either from our end or yours.
It will be closed if no further activity occurs, please write a comment if you would like to keep this going.

Note: in the past months we have built a new client, that has just landed in master. If you want to open an issue or a pr for the legacy client, you should do that in https://github.com/elastic/elasticsearch-js-legacy

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@stale stale bot added stale and removed stale labels Oct 25, 2019
@stale
Copy link

stale bot commented Nov 9, 2019

We understand that this might be important for you, but this issue has been automatically marked as stale because it has not had recent activity either from our end or yours.
It will be closed if no further activity occurs, please write a comment if you would like to keep this going.

Note: in the past months we have built a new client, that has just landed in master. If you want to open an issue or a pr for the legacy client, you should do that in https://github.com/elastic/elasticsearch-js-legacy

@stale stale bot added the stale label Nov 9, 2019
@delvedor delvedor removed the stale label Nov 12, 2019
@stale
Copy link

stale bot commented Nov 24, 2019

We understand that this might be important for you, but this issue has been automatically marked as stale because it has not had recent activity either from our end or yours.
It will be closed if no further activity occurs, please write a comment if you would like to keep this going.

Note: in the past months we have built a new client, that has just landed in master. If you want to open an issue or a pr for the legacy client, you should do that in https://github.com/elastic/elasticsearch-js-legacy

@stale stale bot added the stale label Nov 24, 2019
@delvedor delvedor removed the stale label Nov 25, 2019
@stale
Copy link

stale bot commented Dec 9, 2019

We understand that this might be important for you, but this issue has been automatically marked as stale because it has not had recent activity either from our end or yours.
It will be closed if no further activity occurs, please write a comment if you would like to keep this going.

Note: in the past months we have built a new client, that has just landed in master. If you want to open an issue or a pr for the legacy client, you should do that in https://github.com/elastic/elasticsearch-js-legacy

@stale stale bot added the stale label Dec 9, 2019
@delvedor delvedor removed the stale label Dec 13, 2019
@delvedor delvedor merged commit 59559f7 into elastic:master Dec 19, 2019
delvedor pushed a commit that referenced this pull request Dec 19, 2019
If called without a callback, the request method returns a Promise, so
when calling into super.request, the result should be returned to
maintain promise behavior.
delvedor pushed a commit that referenced this pull request Dec 19, 2019
If called without a callback, the request method returns a Promise, so
when calling into super.request, the result should be returned to
maintain promise behavior.
delvedor pushed a commit that referenced this pull request Dec 19, 2019
If called without a callback, the request method returns a Promise, so
when calling into super.request, the result should be returned to
maintain promise behavior.
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.

4 participants