Skip to content
This repository has been archived by the owner on Feb 1, 2022. It is now read-only.

add wait #32

Merged
merged 5 commits into from
May 31, 2018
Merged

add wait #32

merged 5 commits into from
May 31, 2018

Conversation

RasPhilCo
Copy link
Contributor

@codecov
Copy link

codecov bot commented May 31, 2018

Codecov Report

Merging #32 into master will decrease coverage by 0.55%.
The diff coverage is 25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   46.17%   45.61%   -0.56%     
==========================================
  Files          12       13       +1     
  Lines         327      331       +4     
  Branches       71       71              
==========================================
  Hits          151      151              
- Misses        149      153       +4     
  Partials       27       27
Impacted Files Coverage Δ
src/wait.ts 0% <0%> (ø)
src/deps.ts 55.55% <0%> (-3.27%) ⬇️
src/index.ts 61.19% <100%> (-0.93%) ⬇️

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 7df8fb4...1b087a1. Read the comment docs.

src/deps.ts Outdated
@@ -22,6 +22,7 @@ export const deps = {
get styledHeader(): typeof styledHeader.default { return fetch('./styled/header').default },
get styledJSON(): typeof styledJSON.default { return fetch('./styled/json').default },
get table(): typeof table.default { return fetch('./styled/table').default },
get wait(): typeof table.default { return fetch('./wait').default },
Copy link
Contributor

Choose a reason for hiding this comment

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

this type is wrong. I think you might be able to use the new import() syntax to make this better in typescript 2.9 too

Copy link
Contributor

@jdx jdx left a comment

Choose a reason for hiding this comment

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

would you mind also adding this to the README?

@RasPhilCo RasPhilCo merged commit 5abcfa6 into master May 31, 2018
@RasPhilCo RasPhilCo deleted the wait branch May 31, 2018 22:53
oclif-bot added a commit that referenced this pull request May 31, 2018
<a name="4.5.0"></a>
# [4.5.0](v4.4.1...v4.5.0) (2018-05-31)

### Features

* add wait ([#32](#32)) ([5abcfa6](5abcfa6))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants