Skip to content

Add awaitable option for flush - #3181

Closed
alainrk wants to merge 1 commit into
elastic:mainfrom
alainrk:2857-flush-async
Closed

Add awaitable option for flush#3181
alainrk wants to merge 1 commit into
elastic:mainfrom
alainrk:2857-flush-async

Conversation

@alainrk

@alainrk alainrk commented Mar 2, 2023

Copy link
Copy Markdown

As per #2857 add promise return if no callback is given.

Checklist

  • Implement code
  • Add tests
  • Update TypeScript typings
  • Update documentation
  • Add CHANGELOG.asciidoc entry
  • Commit message follows commit guidelines

@cla-checker-service

Copy link
Copy Markdown

❌ Author of the following commits did not sign a Contributor Agreement:
0b9170a

Please, read and sign the above mentioned agreement if you want to contribute to this project

@github-actions github-actions Bot added agent-nodejs Make available for APM Agents project planning. community triage labels Mar 2, 2023
@trentm

trentm commented Mar 2, 2023

Copy link
Copy Markdown
Member

@alainrk Thanks very much for the PR! I appreciate adding a test case from the get go.

We recently merged a PR for the same issue.

@trentm trentm closed this Mar 2, 2023
@trentm trentm removed the triage label Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-nodejs Make available for APM Agents project planning. community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants