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.

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

refactor: add typings for tts/v1 #968

Merged
merged 2 commits into from
Sep 23, 2019
Merged

refactor: add typings for tts/v1 #968

merged 2 commits into from
Sep 23, 2019

Conversation

MasterOdin
Copy link
Contributor

Adds typings for text-to-speech/v1.ts, using a similar approach as the speech-to-text/v1.ts typings in that the Stream Options are split across the two files as appropriate.

Checklist
  • npm test passes (tip: npm run autofix can correct most style issues)

Copy link
Contributor

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

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

Looks great! Just leaving one nitpicky comment and one ask for a change in the STT code since I like what you did here

lib/synthesize-stream.ts Outdated Show resolved Hide resolved
text-to-speech/v1.ts Show resolved Hide resolved
@codecov-io
Copy link

codecov-io commented Sep 23, 2019

Codecov Report

Merging #968 into release-v5-rc2 will increase coverage by 11.11%.
The diff coverage is n/a.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           release-v5-rc2     #968       +/-   ##
===================================================
+ Coverage           77.77%   88.88%   +11.11%     
===================================================
  Files                   1        1               
  Lines                   9        9               
  Branches                2        2               
===================================================
+ Hits                    7        8        +1     
+ Misses                  2        1        -1
Impacted Files Coverage Δ
test/resources/auth_helper.js 88.88% <0%> (+11.11%) ⬆️

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 8a508bb...e0a1424. Read the comment docs.

Copy link
Contributor

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks again! 👍

@dpopp07 dpopp07 merged commit 7c27052 into watson-developer-cloud:release-v5-rc2 Sep 23, 2019
@MasterOdin MasterOdin deleted the type_tts branch September 27, 2019 12:00
dpopp07 pushed a commit that referenced this pull request Oct 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants