-
Notifications
You must be signed in to change notification settings - Fork 213
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
fix(docs): configuration.copy link #709
Conversation
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
@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. |
@stephenplusplus Thanks for the review, I made changes in one commit manually. Please take a look. |
@laljikanjareeya thank you! I approved, although there's a few quick formatting things. |
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>
Fixes #708