Codecov Report

Merging #1144 (81a9b13) into main (a769e86) will decrease coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1144      +/-   ##
==========================================
- Coverage   66.26%   65.99%   -0.27%     
==========================================
  Files         265      261       -4     
  Lines       16682    16482     -200     
  Branches      157      157              
==========================================
- Hits        11054    10877     -177     
+ Misses       4483     4470      -13     
+ Partials     1145     1135      -10     

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 a769e86...81a9b13. Read the comment docs.

I wonder if this could be autogenned? I've seen that on other repos before, but I'm having trouble finding them!

@kylecarbs I like that idea! I've seen things like actions/contributors-list which displays the users picture/name.

Are we ok with utilizing GitHub Actions for this? And do we want to include the extended information like first PR date and organization? The former may be queryable via GitHub API (need to verify), but as for the latter the user could be part of multiple organizations.

@mafredri I was thinking we'd use something like: https://contrib.rocks/preview?repo=coder%2Fcode-server

Ah, that's actually a pretty nice solution to avoid having to re-generate the README. I suppose we would need a self-hosted solution to access the private repo? One downside with the image version is that you can't click to the users profile (if we consider that important).

That's true. I'd be in favor of using contributors-list in that case, I'm concerned that the manually maintained list will not be manually maintained 😁