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

Pull requests report (2/12/2024 13:31) #86

Open
github-actions bot opened this issue Dec 2, 2024 · 3 comments
Open

Pull requests report (2/12/2024 13:31) #86

github-actions bot opened this issue Dec 2, 2024 · 3 comments

Comments

@github-actions
Copy link

github-actions bot commented Dec 2, 2024

Pull Request report total

This report based on 54 last updated PRs. To learn more about the project and its configuration, please visit Pull request analytics action.

Below are the settings applied for this report:

GITHUB_OWNERS_REPOS: cowprotocol/services
GITHUB_OWNER_FOR_ISSUE: cowprotocol
GITHUB_REPO_FOR_ISSUE: pm
SHOW_STATS_TYPES: timeline, workload, pr-quality, code-review-engagement, response-time
TOP_LIST_AMOUNT: 5
AGGREGATE_VALUE_METHODS: percentile
SHOW_CORRELATION_GRAPHS: true
SHOW_ACTIVITY_TIME_GRAPHS: true
PERCENTILE: 75
TIMEZONE: Europe/Berlin
CORE_HOURS_START: 9:00
CORE_HOURS_END: 19:00
REPORT_DATE_START: 01/10/2024
REPORT_DATE_END: 01/11/2024
AMOUNT: 100
PERIOD_SPLIT_UNIT: months
USE_CHARTS: true
EXECUTION_OUTCOME: markdown, new-issue

References

gantt
title Pull requests timeline(75th percentile) total / minutes
dateFormat X
axisFormat %s
section m-lord-renkse
                Time in draft(21 minutes) :  0, 21
Time to review request(20 minutes) :  0, 20
Time to review(1 hour 49 minutes) :  0, 109
Time to approve(17 hours 11 minutes) :  0, 1031
Time to merge(19 hours 28 minutes) :  0, 1168
        
section MartinquaXD
                Time to review(3 hours 21 minutes) :  0, 201
Time to approve(3 hours 21 minutes) :  0, 201
Time to merge(20 hours 14 minutes) :  0, 1214
        
section mstrug
                Time in draft(10 hours 15 minutes) :  0, 615
Time to review request(10 hours 15 minutes) :  0, 615
Time to review(2 hours 18 minutes) :  0, 138
Time to approve(14 hours 36 minutes) :  0, 876
Time to merge(23 hours 55 minutes) :  0, 1435
        
section squadgazzz
                Time in draft(16 minutes) :  0, 16
Time to review request(16 minutes) :  0, 16
Time to review(4 hours 9 minutes) :  0, 249
Time to approve(6 hours 38 minutes) :  0, 398
Time to merge(7 hours 44 minutes) :  0, 464
        
section sunce86
                Time to review(4 hours 42 minutes) :  0, 282
Time to approve(8 hours 16 minutes) :  0, 496
Time to merge(19 hours 45 minutes) :  0, 1185
        
section total
                Time in draft(2 minutes) :  0, 2
Time to review request(1 minute) :  0, 1
Time to review(4 hours 38 minutes) :  0, 278
Time to approve(10 hours 20 minutes) :  0, 620
Time to merge(19 hours 40 minutes) :  0, 1180
        
Loading

longest-pending reviews

  1. Encode /quote response JIT orders and pre-interactions(52 hours 45 minutes)
  2. Fix liquidity orders observation(43 hours 21 minutes)
  3. Use quote id just for debugging(31 hours 31 minutes)
  4. New database tables for auction and solver competition(22 hours 53 minutes)
  5. Change the e2e tests to be sync-to-blockchain(14 hours 14 minutes)

longest-pending approvals

  1. New database tables for auction and solver competition(188 hours 38 minutes)
  2. Multiple winners in autopilot(129 hours 19 minutes)
  3. Playground check(38 hours 10 minutes)
  4. Driver: Include non-surplus-capturing-jit-orders in the driver /solve(20 hours 52 minutes)
  5. CoW AMM: Implement archive node URL as an option to build CoW AMM Web3(15 hours 13 minutes)

longest-pending merge

  1. Support tips for public mempool submission(44 hours 35 minutes)
  2. New database tables for auction and solver competition(37 hours 39 minutes)
  3. Use blocks to determine other environment settlements(26 hours 56 minutes)
  4. Playground check(17 hours 51 minutes)
  5. Limit number of accepted proposed solutions(16 hours 16 minutes)

Contribution stats total

Reviews conducted - number of reviews conducted. 1 PR may have only single review.
PR Size - determined using the formula: additions + deletions * 0.2. Based on this calculation: 0-50: xs, 51-200: s, 201-400: m, 401-700: l, 701+: xl
Total reverted PRs - The number of reverted PRs based on the branch name pattern /^revert-d+/. This pattern is used for reverts made via GitHub.

user Total opened PRs Total merged PRs Total reverted PRs PRs w/o review PRs w/o approval Additions / Deletions PR size: xs/s/m/l/xl
m-lord-renkse 9 7 0 1 2 +1126/-672 2/5/1/1/0
MartinquaXD 14 13 0 2 2 +749/-695 10/2/2/0/0
mstrug 3 3 0 0 0 +249/-3 2/0/1/0/0
squadgazzz 8 5 0 3 4 +1564/-203 4/0/3/0/1
sunce86 20 18 0 0 1 +2745/-1228 6/9/3/2/0
total 54 46 0 6 9 +6433/-2801 24/16/10/3/1

The largest PRs

  1. Encode /quote response JIT orders and pre-interactions(+786/-48)
  2. New database tables for auction and solver competition(+648/-18)
  3. Unify networks and add Base support(+426/-318)
  4. Revert "Use quote id just for debugging (#3054)"(+391/-306)
  5. Driver /quote response with JIT orders(+334/-19)

Pull request quality total

Agreed - discussions with at least 1 reaction 👍.
Disagreed - discussions with at least 1 reaction 👎.

user Total merged PRs Changes requested received Agreed / Disagreed / Total discussions received Comments received
m-lord-renkse 7 0 26 / 0 / 46 66
MartinquaXD 13 0 3 / 0 / 9 9
mstrug 3 0 4 / 0 / 17 17
squadgazzz 5 0 2 / 0 / 31 41
sunce86 18 0 38 / 0 / 110 148
total 46 0 73 / 0 / 213 281

The most commented PRs

  1. New database tables for auction and solver competition(94)
  2. Multiple winners in autopilot(52)
  3. Unify networks and add Base support(38)
  4. Encode /quote response JIT orders and pre-interactions(38)
  5. Driver: Include non-surplus-capturing-jit-orders in the driver /solve(34)

Code review engagement total

PR Size - determined using the formula: additions + deletions * 0.2. Based on this calculation: 0-50: xs, 51-200: s, 201-400: m, 401-700: l, 701+: xl
Changes requested / Comments / Approvals - number of reviews conducted by user. For a single pull request, only one review of each status will be counted for a user.
Agreed - discussions with at least 1 reaction 👍.
Disagreed - discussions with at least 1 reaction 👎.

user Reviews conducted Agreed / Disagreed / Total discussions conducted Comments conducted PR size: xs/s/m/l/xl Changes requested / Commented / Approved
fhenneke 0 0 / 0 / 0 2 0/0/0/0/0 0 / 0 / 0
fleupold 3 1 / 0 / 12 19 0/1/1/1/0 0 / 3 / 0
m-lord-renkse 29 6 / 0 / 30 34 13/9/5/1/1 0 / 12 / 24
MartinquaXD 35 45 / 0 / 122 164 10/14/7/3/1 0 / 19 / 30
mstrug 1 0 / 0 / 1 1 0/0/1/0/0 0 / 1 / 0
squadgazzz 33 14 / 0 / 30 36 12/12/6/3/0 0 / 16 / 26
sunce86 14 7 / 0 / 18 25 7/4/2/1/0 0 / 3 / 14
total 51 73 / 0 / 213 281 24/16/10/3/1 0 / 29 / 48

Review Response Time(75th percentile) total

Time from re-request to response - time from a review re-request to the response. Multiple re-requests and responses can occur in a single pull request

user Review requests conducted Reviews conducted Time from opening to response Time from initial request to response Time from re-request to response
backend 52 0
fhenneke 1 0
fleupold 2 3 13 hours 13 minutes
m-lord-renkse 6 29 15 hours 7 minutes 1 hour 51 minutes
MartinquaXD 9 35 12 hours 53 minutes 5 hours 12 minutes
mstrug 0 1
squadgazzz 8 33 7 hours 46 minutes 1 hour 1 minute
sunce86 2 14 12 hours 6 minutes 2 hours 56 minutes
total 80 51 13 hours 32 minutes 12 hours 18 minutes 4 hours 8 minutes
Copy link
Author

github-actions bot commented Dec 2, 2024

References

$$\color{gold}Time\ To\ Review\ \color{chartreuse}Time\ To\ Approve\ \color{blueviolet}Time\ To\ Merge$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "gold, chartreuse, blueviolet"
---
xychart-beta
    title "Pull request's time/size graph(75th percentile) m-lord-renkse"
    x-axis ["xs", "s", "m", "l", "xl"]
    y-axis "hours" 0 --> 28
    line [0, 1.73, 0, 0, 0]
line [0, 20.85, 0, 0, 0]
line [0, 27.17, 0, 0, 0]
Loading

$$\color{gold}Time\ To\ Review\ \color{chartreuse}Time\ To\ Approve\ \color{blueviolet}Time\ To\ Merge$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "gold, chartreuse, blueviolet"
---
xychart-beta
    title "Pull request's time/size graph(75th percentile) MartinquaXD"
    x-axis ["xs", "s", "m", "l", "xl"]
    y-axis "hours" 0 --> 35
    line [1.47, 3.62, 22.48, 0, 0]
line [1.47, 4.68, 22.55, 0, 0]
line [5.37, 34.97, 31.83, 0, 0]
Loading

$$\color{gold}Time\ To\ Review\ \color{chartreuse}Time\ To\ Approve\ \color{blueviolet}Time\ To\ Merge$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "gold, chartreuse, blueviolet"
---
xychart-beta
    title "Pull request's time/size graph(75th percentile) mstrug"
    x-axis ["xs", "s", "m", "l", "xl"]
    y-axis "hours" 0 --> 7
    line [0.97, 0, 0, 0, 0]
line [3.25, 0, 0, 0, 0]
line [6.77, 0, 0, 0, 0]
Loading

$$\color{gold}Time\ To\ Review\ \color{chartreuse}Time\ To\ Approve\ \color{blueviolet}Time\ To\ Merge$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "gold, chartreuse, blueviolet"
---
xychart-beta
    title "Pull request's time/size graph(75th percentile) squadgazzz"
    x-axis ["xs", "s", "m", "l", "xl"]
    y-axis "hours" 0 --> 15
    line [0.1, 0, 6.92, 0, 0]
line [0.1, 0, 14.1, 0, 0]
line [3.42, 0, 0, 0, 0]
Loading

$$\color{gold}Time\ To\ Review\ \color{chartreuse}Time\ To\ Approve\ \color{blueviolet}Time\ To\ Merge$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "gold, chartreuse, blueviolet"
---
xychart-beta
    title "Pull request's time/size graph(75th percentile) sunce86"
    x-axis ["xs", "s", "m", "l", "xl"]
    y-axis "hours" 0 --> 129
    line [3.08, 3.8, 13.92, 11.45, 0]
line [3.45, 7.58, 65.65, 107.47, 0]
line [14.93, 16.87, 78.27, 128.35, 0]
Loading

$$\color{gold}Time\ To\ Review\ \color{chartreuse}Time\ To\ Approve\ \color{blueviolet}Time\ To\ Merge$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "gold, chartreuse, blueviolet"
---
xychart-beta
    title "Pull request's time/size graph(75th percentile) total"
    x-axis ["xs", "s", "m", "l", "xl"]
    y-axis "hours" 0 --> 73
    line [1.47, 4.62, 13.2, 7.73, 0]
line [2.27, 11.58, 43.15, 57.85, 0]
line [10.37, 27.53, 59.48, 72.92, 0]
Loading

Copy link
Author

github-actions bot commented Dec 2, 2024

References

$$\color{black}Opened\ \color{purple}Merged\ \color{green}Approved\ \color{red}Changes Requested\ \color{blue}Commented$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "black, purple, green, red, blue"
---
xychart-beta
    title "Activity fleupold"
    x-axis ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23"]
    y-axis "Amount" 0 --> 1
    line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 0, 0, 0, 1, 0, 1, 0, 0, 1, 0, 0, 0]
Loading

$$\color{black}Opened\ \color{purple}Merged\ \color{green}Approved\ \color{red}Changes Requested\ \color{blue}Commented$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "black, purple, green, red, blue"
---
xychart-beta
    title "Activity m-lord-renkse"
    x-axis ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23"]
    y-axis "Amount" 0 --> 9
    line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 0, 2, 1, 2, 2, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 2, 1, 1, 0, 0, 0, 0, 2, 1, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 2, 6, 1, 1, 3, 3, 2, 5, 1, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 6, 2, 7, 7, 4, 1, 9, 9, 0, 1, 0, 0, 0, 0, 0]
Loading

$$\color{black}Opened\ \color{purple}Merged\ \color{green}Approved\ \color{red}Changes Requested\ \color{blue}Commented$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "black, purple, green, red, blue"
---
xychart-beta
    title "Activity MartinquaXD"
    x-axis ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23"]
    y-axis "Amount" 0 --> 8
    line [1, 0, 0, 0, 0, 0, 0, 0, 1, 2, 1, 1, 1, 1, 2, 1, 0, 0, 0, 2, 0, 0, 1, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 1, 2, 2, 1, 2, 0, 3, 1, 1, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 3, 1, 4, 2, 1, 8, 4, 1, 5, 1, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 1, 6, 2, 4, 2, 4, 1, 2, 2, 0, 0, 0, 1, 0, 0, 0]
Loading

$$\color{black}Opened\ \color{purple}Merged\ \color{green}Approved\ \color{red}Changes Requested\ \color{blue}Commented$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "black, purple, green, red, blue"
---
xychart-beta
    title "Activity mstrug"
    x-axis ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23"]
    y-axis "Amount" 0 --> 2
    line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 2]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0]
Loading

$$\color{black}Opened\ \color{purple}Merged\ \color{green}Approved\ \color{red}Changes Requested\ \color{blue}Commented$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "black, purple, green, red, blue"
---
xychart-beta
    title "Activity squadgazzz"
    x-axis ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23"]
    y-axis "Amount" 0 --> 6
    line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 1, 0, 0, 1, 1, 1, 0, 3, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 1, 1, 0, 0, 1, 0, 0, 1, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 3, 1, 1, 1, 2, 2, 3, 5, 3, 0, 1, 3, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 2, 2, 3, 5, 6, 1, 1, 4, 5, 3, 2, 0, 0, 0, 0]
Loading

$$\color{black}Opened\ \color{purple}Merged\ \color{green}Approved\ \color{red}Changes Requested\ \color{blue}Commented$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "black, purple, green, red, blue"
---
xychart-beta
    title "Activity sunce86"
    x-axis ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23"]
    y-axis "Amount" 0 --> 6
    line [1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 3, 1, 5, 2, 2, 2, 1, 0, 0, 1, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 3, 2, 0, 0, 2, 2, 6, 2, 1, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 5, 0, 0, 0, 0, 2, 2, 1, 0, 0, 1, 2, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 2, 3, 1, 2, 0, 1, 0, 3, 0, 0, 0, 0, 0, 0, 0]
Loading

$$\color{black}Opened\ \color{purple}Merged\ \color{green}Approved\ \color{red}Changes Requested\ \color{blue}Commented$$

---
config:
    xyChart:
        width: 900
        height: 600
    themeVariables:
        xyChart:
            titleColor: "black"
            plotColorPalette: "black, purple, green, red, blue"
---
xychart-beta
    title "Activity total"
    x-axis ["0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23"]
    y-axis "Amount" 0 --> 19
    line [2, 0, 0, 0, 0, 0, 0, 0, 1, 2, 4, 3, 4, 4, 9, 5, 4, 3, 3, 3, 0, 4, 1, 2]
line [0, 0, 0, 0, 0, 0, 0, 0, 1, 8, 7, 3, 3, 2, 5, 8, 5, 2, 1, 0, 0, 0, 0, 1]
line [0, 0, 0, 0, 0, 0, 0, 0, 6, 9, 13, 4, 5, 12, 8, 10, 11, 7, 3, 0, 2, 5, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
line [0, 0, 0, 0, 0, 0, 0, 0, 1, 17, 10, 16, 16, 14, 4, 13, 19, 6, 4, 2, 2, 0, 0, 0]
Loading

Copy link
Author

github-actions bot commented Dec 2, 2024

Pull Request report 10/2024

References

gantt
title Pull requests timeline(75th percentile) 10/2024 / minutes
dateFormat X
axisFormat %s
section m-lord-renkse
                Time in draft(21 minutes) :  0, 21
Time to review request(20 minutes) :  0, 20
Time to review(1 hour 49 minutes) :  0, 109
Time to approve(17 hours 11 minutes) :  0, 1031
Time to merge(19 hours 28 minutes) :  0, 1168
        
section MartinquaXD
                Time to review(3 hours 21 minutes) :  0, 201
Time to approve(3 hours 21 minutes) :  0, 201
Time to merge(20 hours 14 minutes) :  0, 1214
        
section mstrug
                Time in draft(10 hours 15 minutes) :  0, 615
Time to review request(10 hours 15 minutes) :  0, 615
Time to review(2 hours 18 minutes) :  0, 138
Time to approve(14 hours 36 minutes) :  0, 876
Time to merge(23 hours 55 minutes) :  0, 1435
        
section squadgazzz
                Time in draft(16 minutes) :  0, 16
Time to review request(16 minutes) :  0, 16
Time to review(4 hours 9 minutes) :  0, 249
Time to approve(6 hours 38 minutes) :  0, 398
Time to merge(7 hours 44 minutes) :  0, 464
        
section sunce86
                Time to review(4 hours 42 minutes) :  0, 282
Time to approve(8 hours 16 minutes) :  0, 496
Time to merge(19 hours 45 minutes) :  0, 1185
        
section total
                Time in draft(2 minutes) :  0, 2
Time to review request(1 minute) :  0, 1
Time to review(4 hours 38 minutes) :  0, 278
Time to approve(10 hours 20 minutes) :  0, 620
Time to merge(19 hours 40 minutes) :  0, 1180
        
Loading

longest-pending reviews

  1. Encode /quote response JIT orders and pre-interactions(52 hours 45 minutes)
  2. Fix liquidity orders observation(43 hours 21 minutes)
  3. Use quote id just for debugging(31 hours 31 minutes)
  4. New database tables for auction and solver competition(22 hours 53 minutes)
  5. Change the e2e tests to be sync-to-blockchain(14 hours 14 minutes)

longest-pending approvals

  1. New database tables for auction and solver competition(188 hours 38 minutes)
  2. Multiple winners in autopilot(129 hours 19 minutes)
  3. Playground check(38 hours 10 minutes)
  4. Driver: Include non-surplus-capturing-jit-orders in the driver /solve(20 hours 52 minutes)
  5. CoW AMM: Implement archive node URL as an option to build CoW AMM Web3(15 hours 13 minutes)

longest-pending merge

  1. Support tips for public mempool submission(44 hours 35 minutes)
  2. New database tables for auction and solver competition(37 hours 39 minutes)
  3. Use blocks to determine other environment settlements(26 hours 56 minutes)
  4. Playground check(17 hours 51 minutes)
  5. Limit number of accepted proposed solutions(16 hours 16 minutes)

Contribution stats 10/2024

Reviews conducted - number of reviews conducted. 1 PR may have only single review.
PR Size - determined using the formula: additions + deletions * 0.2. Based on this calculation: 0-50: xs, 51-200: s, 201-400: m, 401-700: l, 701+: xl
Total reverted PRs - The number of reverted PRs based on the branch name pattern /^revert-d+/. This pattern is used for reverts made via GitHub.

user Total opened PRs Total merged PRs Total reverted PRs PRs w/o review PRs w/o approval Additions / Deletions PR size: xs/s/m/l/xl
m-lord-renkse 9 7 0 1 2 +1126/-672 2/5/1/1/0
MartinquaXD 14 13 0 2 2 +749/-695 10/2/2/0/0
mstrug 3 3 0 0 0 +249/-3 2/0/1/0/0
squadgazzz 8 5 0 3 4 +1564/-203 4/0/3/0/1
sunce86 20 18 0 0 1 +2745/-1228 6/9/3/2/0
total 54 46 0 6 9 +6433/-2801 24/16/10/3/1

The largest PRs

  1. Encode /quote response JIT orders and pre-interactions(+786/-48)
  2. New database tables for auction and solver competition(+648/-18)
  3. Unify networks and add Base support(+426/-318)
  4. Revert "Use quote id just for debugging (#3054)"(+391/-306)
  5. Driver /quote response with JIT orders(+334/-19)

Pull request quality 10/2024

Agreed - discussions with at least 1 reaction 👍.
Disagreed - discussions with at least 1 reaction 👎.

user Total merged PRs Changes requested received Agreed / Disagreed / Total discussions received Comments received
m-lord-renkse 7 0 26 / 0 / 46 66
MartinquaXD 13 0 3 / 0 / 9 9
mstrug 3 0 4 / 0 / 17 17
squadgazzz 5 0 2 / 0 / 31 41
sunce86 18 0 38 / 0 / 110 148
total 46 0 73 / 0 / 213 281

The most commented PRs

  1. New database tables for auction and solver competition(94)
  2. Multiple winners in autopilot(52)
  3. Unify networks and add Base support(38)
  4. Encode /quote response JIT orders and pre-interactions(38)
  5. Driver: Include non-surplus-capturing-jit-orders in the driver /solve(34)

Code review engagement 10/2024

PR Size - determined using the formula: additions + deletions * 0.2. Based on this calculation: 0-50: xs, 51-200: s, 201-400: m, 401-700: l, 701+: xl
Changes requested / Comments / Approvals - number of reviews conducted by user. For a single pull request, only one review of each status will be counted for a user.
Agreed - discussions with at least 1 reaction 👍.
Disagreed - discussions with at least 1 reaction 👎.

user Reviews conducted Agreed / Disagreed / Total discussions conducted Comments conducted PR size: xs/s/m/l/xl Changes requested / Commented / Approved
fhenneke 0 0 / 0 / 0 2 0/0/0/0/0 0 / 0 / 0
fleupold 3 1 / 0 / 12 19 0/1/1/1/0 0 / 3 / 0
m-lord-renkse 29 6 / 0 / 30 34 13/9/5/1/1 0 / 12 / 24
MartinquaXD 35 45 / 0 / 122 164 10/14/7/3/1 0 / 19 / 30
mstrug 1 0 / 0 / 1 1 0/0/1/0/0 0 / 1 / 0
squadgazzz 33 14 / 0 / 30 36 12/12/6/3/0 0 / 16 / 26
sunce86 14 7 / 0 / 18 25 7/4/2/1/0 0 / 3 / 14
total 51 73 / 0 / 213 281 24/16/10/3/1 0 / 29 / 48

Review Response Time(75th percentile) 10/2024

Time from re-request to response - time from a review re-request to the response. Multiple re-requests and responses can occur in a single pull request

user Review requests conducted Reviews conducted Time from opening to response Time from initial request to response Time from re-request to response
backend 52 0
fhenneke 1 0
fleupold 2 3 13 hours 13 minutes
m-lord-renkse 6 29 15 hours 7 minutes 1 hour 51 minutes
MartinquaXD 9 35 12 hours 53 minutes 5 hours 12 minutes
mstrug 0 1
squadgazzz 8 33 7 hours 46 minutes 1 hour 1 minute
sunce86 2 14 12 hours 6 minutes 2 hours 56 minutes
total 80 51 13 hours 32 minutes 12 hours 18 minutes 4 hours 8 minutes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

0 participants