-
Notifications
You must be signed in to change notification settings - Fork 6.3k
docs: Fix links #4770
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
docs: Fix links #4770
Conversation
Codecov Report
Merging #4770 (b8a6f76) into main (51818dc) will not change coverage.
The diff coverage isn/a.
@@ Coverage Diff @@ ## main #4770 +/- ## ======================================= Coverage 69.18% 69.18% ======================================= Files 29 29 Lines 1652 1652 Branches 363 363 ======================================= Hits 1143 1143 Misses 432 432 Partials 77 77
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 51818dc...b8a6f76. Read the comment docs.
Also, an unrelated question - how did you enable Codecov?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for doing this!
* docs: Change `cdr` to `coder`, read desc please Yeah, includes links. * docs: Update README (links and the extra comma) Co-authored-by: Joe Previte <jjprevite@gmail.com>
Fixes #
Minor fix
@jsjoeio I have applied the changes(
cdrtocoder) to the links, user(in this case, org) and removed the extra comma in the README(code-server or Coder difference) that made me think it's not right.This is ready for merge, however, should I run
yarn fmtand rebase(before FMT of course) again?