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

fix(docs): configuration.copy link #709

Merged

Conversation

laljikanjareeya
Copy link
Contributor

Fixes #708

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 29, 2020
@codecov
Copy link

codecov bot commented May 29, 2020

Codecov Report

Merging #709 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #709      +/-   ##
==========================================
- Coverage   95.14%   95.14%   -0.01%     
==========================================
  Files           7        7              
  Lines        6282     6279       -3     
  Branches      406      406              
==========================================
- Hits         5977     5974       -3     
  Misses        305      305              
Impacted Files Coverage Δ
src/table.ts 99.95% <100.00%> (-0.01%) ⬇️

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 88f40a9...161dbd5. Read the comment docs.

src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
@stephenplusplus
Copy link
Contributor

@laljikanjareeya sorry that my review makes it look like everything is wrong! I just had some ideas for new phrasing, and tried to leave comments with code suggestions so you could just click to accept them and have the code update automatically.

@laljikanjareeya
Copy link
Contributor Author

@stephenplusplus Thanks for the review, I made changes in one commit manually. Please take a look.

src/table.ts Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
src/table.ts Outdated Show resolved Hide resolved
@stephenplusplus
Copy link
Contributor

@laljikanjareeya thank you! I approved, although there's a few quick formatting things.

laljikanjareeya and others added 4 commits June 2, 2020 07:43
Co-authored-by: Stephen <stephenplusplus@users.noreply.github.com>
Co-authored-by: Stephen <stephenplusplus@users.noreply.github.com>
Co-authored-by: Stephen <stephenplusplus@users.noreply.github.com>
Co-authored-by: Stephen <stephenplusplus@users.noreply.github.com>
@laljikanjareeya laljikanjareeya merged commit 4a81b1e into googleapis:master Jun 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect documentation link for configuration.copy in table
4 participants