Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(timers): Implement drop-in replacement of futures_timer::Delay #422

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

oblique
Copy link

@oblique oblique commented Aug 7, 2023

Description

In some cases a compatibility layer with futures-timer is needed. This PR add a drop-in replacement for futures_timer::Delay.

How Has This Been Tested?

Started a local node and tested a simple call to ic_cdk_timers::Delay.

Checklist:

  • The title of this PR complies with Conventional Commits.
  • I have edited the CHANGELOG accordingly.
  • I have made corresponding changes to the documentation.

@ghost
Copy link

ghost commented Aug 7, 2023

Dear @oblique,

In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA1.

If you decide to agree with it, please visit this issue and read the instructions there.

— The DFINITY Foundation

Footnotes

  1. Contributor License Agreement

@ghost ghost added the cla:pending label Aug 7, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant