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.

Sign up for GitHub

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

Jump to bottom

feat: add utils/delay #547

Open
wants to merge 41 commits into
base: develop
Choose a base branch
from
Open

feat: add utils/delay #547

wants to merge 41 commits into from

Conversation

Planeshifter
Copy link
Member

Description

What is the purpose of this pull request?

This pull request:

  • adds a utility to invoke a function after a delay. In contrast to using setTimeout, which returns a timeout identifier that can be used to clear a pending invocation by calling clearTimeout, the delay function returns a timeout object with additional properties and methods.

Related Issues

Does this pull request have any related issues?

This pull request:

  • resolves #
  • fixes #

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

@stdlib-js/reviewers

@github-actions
Copy link
Contributor

Based off the README.md file, the stdlib-bot has generated contents for the delay package.

Notes

  • ⚠️ The stdlib-bot uses AI to generate package content. The generated content is not guaranteed to be correct, and will require manual review and editing. ⚠️

@Planeshifter
Copy link
Member Author

@kgryte Any objections to the API as laid out in the README.md? If not, will implement, clean-up, and push and then ping you for final review.

@kgryte kgryte added the Feature Issue or pull request for adding a new feature. label Oct 15, 2022
@kgryte kgryte added the Needs Changes Pull request which needs changes before being merged. label Feb 23, 2024
@kgryte kgryte changed the title Add utility to invoke a function after a delay feat: add utils/delay Feb 24, 2024
@stdlib-js stdlib-js deleted a comment from stdlib-bot Mar 2, 2024
@stdlib-js stdlib-js deleted a comment from stdlib-bot Mar 2, 2024
@stdlib-js stdlib-js deleted a comment from stdlib-bot Mar 2, 2024
@Planeshifter Planeshifter requested a review from Copilot November 16, 2024 01:28
Copilot
Copilot AI reviewed Nov 16, 2024
View reviewed changes

Choose a reason for hiding this comment

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

Copilot reviewed 10 out of 10 changed files in this pull request and generated no suggestions.
@Planeshifter
Copy link
Member Author

/stdlib merge

@stdlib-bot
Copy link
Contributor

Coverage Report

No coverage information available.

@Planeshifter
Copy link
Member Author

/stdlib rebase

@Planeshifter
Copy link
Member Author

/stdlib merge

@Planeshifter
Copy link
Member Author

/stdlib merge

@kgryte kgryte added the Do Not Merge Pull request which should not be merged. label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot Copilot left review comments

@kgryte kgryte kgryte left review comments

@stdlib-bot stdlib-bot Awaiting requested review from stdlib-bot

At least 1 approving review is required to merge this pull request.

Assignees

@stdlib-bot stdlib-bot

Labels
Do Not Merge Pull request which should not be merged. Feature Issue or pull request for adding a new feature. Needs Changes Pull request which needs changes before being merged. Utilities Issue or pull request concerning general utilities.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Planeshifter @kgryte @stdlib-bot