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.

Sign up for GitHub

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

Jump to bottom

Update Editors, Authors, and Acknowledgements based on latest data. #92

Merged
merged 4 commits into from
Jan 26, 2025
Merged

Update Editors, Authors, and Acknowledgements based on latest data. #92

merged 4 commits into from
Jan 26, 2025

Conversation

msporny
Copy link
Member

@msporny msporny commented Jan 15, 2025 •
edited by pr-preview bot
Loading

This PR finalizes the Acknowledgements, Authors, and Editors lists based on the following data.

For the Editors, their contributions since the beginning of the WG are visible here:

https://github.com/w3c/vc-di-ecdsa/graphs/contributors?from=6%2F1%2F2022

image

For Acknowledgements, the tally of commit comments are here (and contributors to test suites are also included):

https://w3c.github.io/vc-di-ecdsa-test-suite/

            user |  score |   comments |    bytes |
         msporny |    121 |         95 |    26703 |
       Wind4Greg |     43 |         28 |    15695 |
        dlongley |     31 |         21 |    10717 |
         iherman |     24 |         18 |     7165 |
            OR13 |     16 |         12 |     4818 |
         TallTed |     10 |          7 |     3108 |
        bnewbold |      6 |          4 |     2073 |
         filip26 |      6 |          5 |     1451 |
     peacekeeper |      5 |          4 |     1792 |
 brianorwhatever |      4 |          4 |      375 |
         clehner |      2 |          2 |      880 |
      JSAssassin |      2 |          2 |      570 |
       martyr280 |      1 |          1 |       59 |
 dmitrizagidulin |      1 |          1 |      541 |
  seabass-labrax |      1 |          1 |      248 |

Preview | Diff

@msporny msporny requested a review from Wind4Greg as a code owner January 15, 2025 15:58
@msporny msporny added editorial The item is editorial in nature. CR2 This item was processed during the second Candidate Recommendation Phase labels Jan 15, 2025
TallTed
TallTed reviewed Jan 15, 2025
View reviewed changes
@msporny
Copy link
Member Author

msporny commented Jan 26, 2025

Editorial, multiple reviews, changes requested and made, no objections, merging.

@msporny msporny merged commit 9d7f880 into main Jan 26, 2025
2 checks passed
@msporny msporny deleted the msporny-final-ack branch January 26, 2025 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TallTed TallTed TallTed left review comments

@dlongley dlongley dlongley approved these changes

@iherman iherman iherman approved these changes

@Wind4Greg Wind4Greg Wind4Greg approved these changes

@brianorwhatever brianorwhatever brianorwhatever approved these changes

@seabass-labrax seabass-labrax seabass-labrax approved these changes

@BigBlueHat BigBlueHat Awaiting requested review from BigBlueHat

@davidlehn davidlehn Awaiting requested review from davidlehn

@martyr280 martyr280 Awaiting requested review from martyr280 martyr280 is a code owner

@dmitrizagidulin dmitrizagidulin Awaiting requested review from dmitrizagidulin

@brentzundel brentzundel Awaiting requested review from brentzundel

Assignees
No one assigned
Labels
CR2 This item was processed during the second Candidate Recommendation Phase editorial The item is editorial in nature.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@msporny @dlongley @iherman @Wind4Greg @TallTed @brianorwhatever @seabass-labrax