Skip to content
This repository was archived by the owner on Mar 11, 2026. It is now read-only.

chore(typescript): convert src/ to typescript#258

Merged
JustinBeckwith merged 3 commits intogoogleapis:masterfrom
JustinBeckwith:ts1
Oct 23, 2018
Merged

chore(typescript): convert src/ to typescript#258
JustinBeckwith merged 3 commits intogoogleapis:masterfrom
JustinBeckwith:ts1

Conversation

@JustinBeckwith
Copy link
Copy Markdown
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 23, 2018
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 23, 2018

Codecov Report

Merging #258 into master will decrease coverage by 6.31%.
The diff coverage is 95.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #258      +/-   ##
==========================================
- Coverage   98.15%   91.84%   -6.32%     
==========================================
  Files          15        9       -6     
  Lines        2497      601    -1896     
  Branches        0       58      +58     
==========================================
- Hits         2451      552    -1899     
+ Misses         46       35      -11     
- Partials        0       14      +14
Impacted Files Coverage Δ
src/entry.ts 89.65% <100%> (ø)
src/index.ts 96.63% <100%> (ø)
src/log.ts 100% <100%> (ø)
src/metadata.ts 100% <100%> (ø)
src/sink.ts 89.18% <80%> (ø)
test/entry.js
... and 5 more

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 7c24a2c...5b6198e. Read the comment docs.

Copy link
Copy Markdown
Contributor

@ofrobots ofrobots left a comment

Choose a reason for hiding this comment

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

I was working on the same thing (but without the noImplicitAny), but you beat me to it.

It is the right call to use noImplicitAny as a first step.

@JustinBeckwith
Copy link
Copy Markdown
Contributor Author

Rats. I went ahead and created tracking issues so we don't step on each other's toes :) . I'll assign stuff to myself before workin on it :)

@jkwlui jkwlui added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 23, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 23, 2018
@JustinBeckwith JustinBeckwith merged commit e2a92b4 into googleapis:master Oct 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants