-
Notifications
You must be signed in to change notification settings - Fork 21
Pull requests report (3/06/2025 12:59) #516
Description
Pull Request report total
This report based on 230 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: jumpstarter-dev/jumpstarter
GITHUB_OWNER_FOR_ISSUE: jumpstarter-dev
GITHUB_REPO_FOR_ISSUE: jumpstarter
SHOW_STATS_TYPES: timeline, workload, pr-quality, code-review-engagement, response-time
TOP_LIST_AMOUNT: 5
AGGREGATE_VALUE_METHODS: percentile
SHOW_CORRELATION_GRAPHS: false
SHOW_ACTIVITY_TIME_GRAPHS: false
PERCENTILE: 75
TIMEZONE: Etc/UTC
WEEKENDS: 0,6
REPORT_DATE_START: 1/1/2025
REPORT_DATE_END: 3/6/2025
AMOUNT: 100
PERIOD_SPLIT_UNIT: months
USE_CHARTS: true
EXECUTION_OUTCOME: new-issue
References
- Pull Request report retrospective timeline
- Pull Request report 5/2025
- Pull Request report 4/2025
- Pull Request report 3/2025
- Pull Request report 2/2025
- Pull Request report 1/2025
gantt
title Pull requests timeline(75th percentile) total / minutes
dateFormat X
axisFormat %s
section ahadas
Time to review(20 hours 57 minutes) : 0, 1257
Time to approve(45 hours 21 minutes) : 0, 2721
Time to merge(45 hours 22 minutes) : 0, 2722
section bennyz
Time to review request(21 minutes) : 0, 21
Time to review(6 minutes) : 0, 6
Time to approve(17 hours 14 minutes) : 0, 1034
Time to merge(17 hours 14 minutes) : 0, 1034
section eballetbo
Time in draft(83 hours 56 minutes) : 0, 5036
Time to review(3 minutes) : 0, 3
Time to approve(144 hours 34 minutes) : 0, 8674
Time to merge(204 hours 16 minutes) : 0, 12256
section jumpstarter-backport-bot[bot]
Time to review(1 hour 19 minutes) : 0, 79
Time to approve(1 hour 19 minutes) : 0, 79
Time to merge(1 hour 19 minutes) : 0, 79
section kirkbrauer
Time in draft(12 hours 1 minute) : 0, 721
Time to review request(10 minutes) : 0, 10
Time to review(25 hours 12 minutes) : 0, 1512
Time to approve(76 hours 27 minutes) : 0, 4587
Time to merge(76 hours 29 minutes) : 0, 4589
section mangelajo
Time to review request(19 hours 14 minutes) : 0, 1154
Time to review(29 minutes) : 0, 29
Time to approve(24 hours 44 minutes) : 0, 1484
Time to merge(26 hours 23 minutes) : 0, 1583
section NickCao
Time to review request(14 hours 54 minutes) : 0, 894
Time to review(1 hour 42 minutes) : 0, 102
Time to approve(26 hours 12 minutes) : 0, 1572
Time to merge(28 hours 7 minutes) : 0, 1687
section odra
Time to review(5 minutes) : 0, 5
Time to approve(115 hours 27 minutes) : 0, 6927
Time to merge(122 hours 57 minutes) : 0, 7377
section raballew
Time to review(7 minutes) : 0, 7
Time to approve(18 hours 10 minutes) : 0, 1090
Time to merge(19 hours 8 minutes) : 0, 1148
section total
Time to review request(11 hours 28 minutes) : 0, 688
Time to review(1 hour 23 minutes) : 0, 83
Time to approve(26 hours 2 minutes) : 0, 1562
Time to merge(27 hours 48 minutes) : 0, 1668
longest-pending reviews
- Shell driver for the exporter(454 hours 9 minutes)(Author: mangelajo)
- Improve PySerialClient interface and documentation(166 hours 39 minutes)(Author: mangelajo)
- Update Documentation to Reflect Refactoring(159 hours 12 minutes)(Author: kirkbrauer)
- Handle all instances of tls credentials(145 hours 3 minutes)(Author: mangelajo)
- Provide a driver scaffold generator(128 hours 44 minutes)(Author: mangelajo)
longest-pending approvals
- Add drivers for EnerGenie Power Management System (PMS) products(577 hours 5 minutes)(Author: eballetbo)
- Add DbusNetwork driver(422 hours 2 minutes)(Author: NickCao)
- corellium driver(333 hours 36 minutes)(Author: odra)
- Implement TasmotaPower driver(178 hours 2 minutes)(Author: NickCao)
- Implement MultipathExporterStub allowing for connecting to exporters …(167 hours 20 minutes)(Author: NickCao)
longest-pending merge
- Shell driver for the exporter(358 hours 24 minutes)(Author: mangelajo)
- Add drivers for EnerGenie Power Management System (PMS) products(238 hours 17 minutes)(Author: eballetbo)
- Provide a driver scaffold generator(85 hours 24 minutes)(Author: mangelajo)
- snmp: initial SNMP driver for power control(73 hours 33 minutes)(Author: bennyz)
- http: simplify test and fix minor issues(68 hours 12 minutes)(Author: bennyz)
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 |
|---|---|---|---|---|---|---|---|
| ahadas | 2 | 2 | 0 | 1 | 1 | +67/-96 | 1/1/0/0/0 |
| bennyz | 20 | 19 | 0 | 6 | 4 | +1964/-724 | 13/4/1/2/0 |
| eballetbo | 3 | 3 | 0 | 0 | 0 | +341/-9 | 2/0/1/0/0 |
| jumpstarter-backport-bot[bot] | 17 | 17 | 0 | 1 | 1 | +3772/-3436 | 10/2/4/0/1 |
| kirkbrauer | 7 | 7 | 0 | 2 | 1 | +7983/-2722 | 2/1/0/1/3 |
| mangelajo | 44 | 41 | 0 | 4 | 4 | +6228/-1202 | 26/8/7/1/2 |
| mickume | 1 | 0 | 0 | 1 | 1 | +68/-4191 | 0/0/0/0/1 |
| NickCao | 123 | 116 | 0 | 6 | 10 | +21548/-14487 | 50/45/14/7/7 |
| odra | 3 | 3 | 0 | 0 | 0 | +1787/-166 | 0/1/0/1/1 |
| raballew | 10 | 9 | 0 | 0 | 1 | +7255/-6381 | 2/3/1/1/3 |
| total | 230 | 217 | 0 | 21 | 23 | +51013/-33414 | 106/65/28/13/18 |
The largest PRs
- Introducing rust to jumpstarter(+3038/-652)(Author: NickCao)
- Overhaul documentation(+2870/-1456)(Author: raballew)
- Re-organize Jumpstarter packages to allow sparse dependency tree per-package(+2553/-1687)(Author: kirkbrauer)
- CLI Enhancements(+2715/-408)(Author: kirkbrauer)
- docs: improve reading flow(+2163/-3052)(Author: raballew)
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 |
|---|---|---|---|---|
| ahadas | 2 | 0 | 0 / 0 / 3 | 6 |
| bennyz | 19 | 2 | 10 / 0 / 58 | 76 |
| eballetbo | 3 | 0 | 3 / 0 / 14 | 18 |
| jumpstarter-backport-bot[bot] | 17 | 0 | 0 / 0 / 0 | 0 |
| kirkbrauer | 7 | 1 | 3 / 0 / 30 | 32 |
| mangelajo | 41 | 1 | 8 / 0 / 72 | 83 |
| NickCao | 116 | 3 | 3 / 1 / 190 | 229 |
| odra | 3 | 0 | 1 / 0 / 48 | 50 |
| raballew | 9 | 2 | 1 / 0 / 50 | 62 |
| total | 217 | 9 | 29 / 1 / 465 | 556 |
pie
title Discussion's types mangelajo total
" $release =~ ^v0-9+\.0-9+\.0-9+\w+$ (1)":1
" $REDIRECT =~ ^v0-9+\.0-9+\.0-9+.*$ (2)":2
" $CURRENT_BRANCH == "main" (2)":2
" $release =~ ^v0-9+\.0-9+\.0-9+.*$ (2)":2
pie
title Discussion's types total total
" $release =~ ^v0-9+\.0-9+\.0-9+\w+$ (1)":1
" $REDIRECT =~ ^v0-9+\.0-9+\.0-9+.*$ (2)":2
" $CURRENT_BRANCH == "main" (2)":2
" $release =~ ^v0-9+\.0-9+\.0-9+.*$ (2)":2
The most commented PRs
- Overhaul documentation(49)(Author: raballew)
- snmp: initial SNMP driver for power control(48)(Author: bennyz)
- corellium driver(41)(Author: odra)
- Flasher driver(29)(Author: mangelajo)
- Consolidate Websites(24)(Author: raballew)
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 |
|---|---|---|---|---|---|
| bennyz | 27 | 0 / 0 / 4 | 4 | 13/6/6/0/2 | 0 / 3 / 26 |
| bkhizgiy | 1 | 0 / 0 / 0 | 0 | 1/0/0/0/0 | 0 / 0 / 1 |
| coderabbitai[bot] | 155 | 8 / 1 / 288 | 327 | 57/50/23/12/13 | 0 / 155 / 0 |
| Copilot | 6 | 0 / 0 / 4 | 4 | 6/0/0/0/0 | 0 / 6 / 0 |
| kirkbrauer | 8 | 3 / 0 / 14 | 18 | 4/2/1/1/0 | 0 / 2 / 6 |
| mangelajo | 152 | 10 / 0 / 131 | 172 | 60/49/19/10/14 | 8 / 47 / 147 |
| NickCao | 45 | 8 / 0 / 21 | 28 | 26/7/6/4/2 | 1 / 14 / 40 |
| raballew | 5 | 0 / 0 / 3 | 3 | 4/0/1/0/0 | 0 / 1 / 3 |
| total | 225 | 29 / 1 / 465 | 556 | 106/65/28/13/18 | 9 / 167 / 216 |
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 |
|---|---|---|---|---|---|
| bennyz | 20 | 27 | 44 hours 2 minutes | 18 hours 3 minutes | |
| bkhizgiy | 1 | 1 | |||
| coderabbitai[bot] | 0 | 155 | 6 minutes | ||
| Copilot | 6 | 6 | 1 minute | ||
| kirkbrauer | 17 | 8 | 8 hours 57 minutes | ||
| mangelajo | 26 | 152 | 18 hours 30 minutes | 21 hours 47 minutes | 75 hours 24 minutes |
| NickCao | 28 | 45 | 7 hours 57 minutes | 5 hours 6 minutes | 1 hour 52 minutes |
| raballew | 7 | 5 | 18 hours 29 minutes | 9 hours 14 minutes | |
| total | 105 | 225 | 13 hours 10 minutes | 17 hours 27 minutes | 4 hours 7 minutes |