Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

feat: add setTimeout function that returns a promise. #223

Merged
merged 1 commit into from
Sep 9, 2019

Conversation

kristw
Copy link
Contributor

@kristw kristw commented Sep 6, 2019

🏆 Enhancements

Add promiseTimeout, a setTimeout function that returns a promise.

@kristw kristw requested a review from a team as a code owner September 6, 2019 01:18
@kristw kristw added the reviewable Ready for review label Sep 6, 2019
@kristw kristw changed the title feat: add promiseTimeout feat: add setTimeout function that returns a promise. Sep 6, 2019
@netlify
Copy link

netlify bot commented Sep 6, 2019

Deploy preview for superset-ui ready!

Built with commit 30ff534

https://deploy-preview-223--superset-ui.netlify.com

@codecov
Copy link

codecov bot commented Sep 6, 2019

Codecov Report

Merging #223 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #223   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files         130    131    +1     
  Lines        1416   1419    +3     
  Branches      358    358           
=====================================
+ Hits         1416   1419    +3
Impacted Files Coverage Δ
...kages/superset-ui-core/src/utils/promiseTimeout.ts 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6661bcd...30ff534. Read the comment docs.

@kristw kristw merged commit d02bb82 into master Sep 9, 2019
@delete-merged-branch delete-merged-branch bot deleted the kristw--promise-timeout branch September 9, 2019 19:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
reviewable Ready for review size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants