Pull requests report(19/08/2024 20:30) #4273
Description
Pull Request report total
This report based on 86 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: bcgov/psp
GITHUB_OWNER_FOR_ISSUE: bcgov
GITHUB_REPO_FOR_ISSUE: psp
SHOW_STATS_TYPES: timeline, workload, pr-quality, code-review-engagement, response-time
TOP_LIST_AMOUNT: 5
AGGREGATE_VALUE_METHODS: percentile
PERCENTILE: 75
REPORT_DATE_START: 1/07/2024
REPORT_DATE_END: 29/07/2024
AMOUNT: 100
PERIOD_SPLIT_UNIT: months
EXECUTION_OUTCOME: new-issue
References
Pull requests timeline(75th percentile) total
Time to review - time from PR creation to first review.
Time to approve - time from PR creation to first approval without requested changes.
Time to merge - time from PR creation to merge.
| user | Time to review | Time to approve | Time to merge | Total merged PRs |
|---|---|---|---|---|
| AG/PSSG - Quartech | 19 hours 18 minutes | 19 hours 18 minutes | 22 hours 50 minutes | 32 |
| ap-quartech | 1 | |||
| asanchezr | 16 hours 42 minutes | 16 hours 42 minutes | 18 hours 19 minutes | 11 |
| devinleighsmith | 19 hours 18 minutes | 19 hours 18 minutes | 22 hours 50 minutes | 32 |
| dfilteau | 22 hours 47 minutes | 22 hours 47 minutes | 22 hours 49 minutes | 5 |
| eddherrera | 1 hour 43 minutes | 9 hours 46 minutes | 18 hours 25 minutes | 14 |
| FuriousLlama | 9 hours 58 minutes | 19 hours 44 minutes | 20 hours 36 minutes | 9 |
| PIMS Developers | 16 hours 42 minutes | 16 hours 42 minutes | 18 hours 19 minutes | 11 |
| stairaku | 2 hours 9 minutes | 2 hours 9 minutes | 3 hours 38 minutes | 9 |
| total | 18 hours 24 minutes | 19 hours 54 minutes | 22 hours 22 minutes | 81 |
gantt
title Pull requests timeline(75th percentile) total / minutes
dateFormat X
axisFormat %s
section AG/PSSG - Quartech
Time to review(19 hours 18 minutes) : 0, 1158
Time to approve(19 hours 18 minutes) : 0, 1158
Time to merge(22 hours 50 minutes) : 0, 1370
section asanchezr
Time to review(16 hours 42 minutes) : 0, 1002
Time to approve(16 hours 42 minutes) : 0, 1002
Time to merge(18 hours 19 minutes) : 0, 1099
section devinleighsmith
Time to review(19 hours 18 minutes) : 0, 1158
Time to approve(19 hours 18 minutes) : 0, 1158
Time to merge(22 hours 50 minutes) : 0, 1370
section dfilteau
Time to review(22 hours 47 minutes) : 0, 1367
Time to approve(22 hours 47 minutes) : 0, 1367
Time to merge(22 hours 49 minutes) : 0, 1369
section eddherrera
Time to review(1 hour 43 minutes) : 0, 103
Time to approve(9 hours 46 minutes) : 0, 586
Time to merge(18 hours 25 minutes) : 0, 1105
section FuriousLlama
Time to review(9 hours 58 minutes) : 0, 598
Time to approve(19 hours 44 minutes) : 0, 1184
Time to merge(20 hours 36 minutes) : 0, 1236
section PIMS Developers
Time to review(16 hours 42 minutes) : 0, 1002
Time to approve(16 hours 42 minutes) : 0, 1002
Time to merge(18 hours 19 minutes) : 0, 1099
section stairaku
Time to review(2 hours 9 minutes) : 0, 129
Time to approve(2 hours 9 minutes) : 0, 129
Time to merge(3 hours 38 minutes) : 0, 218
section total
Time to review(18 hours 24 minutes) : 0, 1104
Time to approve(19 hours 54 minutes) : 0, 1194
Time to merge(22 hours 22 minutes) : 0, 1342
longest-pending reviews
- psp-8783 ensure that form state is maintained during collapsing of sidebar.(118 hours 52 minutes)
- psp-8365 lease flexible duration(81 hours 1 minute)
- psp-8708 fix tenant primary contacts.(67 hours 25 minutes)
- PSP-7158 Show markers positioned corresponding to that file (65 hours 44 minutes)
- PSP-8414 : Select one or multiple properties for H120 Generation(65 hours 2 minutes)
longest-pending approvals
- PSP-8414 : Select one or multiple properties for H120 Generation(26 hours 1 minute)
- psp-8630 | Updated Leases and added renewals(18 hours 46 minutes)
- PSP-8896 : FT:AcquisitionFile:Requisition: System generates warning m…(16 hours 55 minutes)
- PSP-8317: Lease and Licence Edit/View Fee Denomination forms(4 hours 18 minutes)
- PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(2 hours 45 minutes)
longest-pending merge
- psp-8018: display retired warning immediately upon add.(122 hours 48 minutes)
- PSP-8708: Organization Tenants with person contacts are not displayed when tenant is selected. Small change on Contacts List View(75 hours 46 minutes)
- PSP-7158 Show markers positioned corresponding to that file (51 hours 4 minutes)
- psp-8772 lease payment export corrections.(40 hours 52 minutes)
- psp-8545 Correct behaviour when there is no property ownership specif…(21 hours 12 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.5. Based on this calculation: 0-50: xs, 51-200: s, 201-400: m, 401-700: l, 701+: xl
| user | Total opened PRs | Total merged PRs | Additions/Deletions | PR size: xs/s/m/l/xl | Total comments | Reviews conducted |
|---|---|---|---|---|---|---|
| AG/PSSG - Quartech | 35 | 32 | +12692/-4061 | 21/7/2/0/5 | 15 | 26 |
| ap-quartech | 1 | 1 | +2671/-55 | 0/0/0/0/1 | 0 | 0 |
| asanchezr | 11 | 11 | +317326/-51499 | 4/2/2/0/3 | 6 | 39 |
| devinleighsmith | 35 | 32 | +12692/-4061 | 21/7/2/0/5 | 15 | 31 |
| dfilteau | 5 | 5 | +189664/-30353 | 1/1/1/0/2 | 0 | 0 |
| eddherrera | 14 | 14 | +4067/-2436 | 6/2/3/1/2 | 14 | 15 |
| FuriousLlama | 9 | 9 | +2569/-1158 | 3/2/0/3/1 | 36 | 12 |
| PIMS Developers | 11 | 11 | +317326/-51499 | 4/2/2/0/3 | 6 | 38 |
| stairaku | 11 | 9 | +1771/-586 | 6/3/0/1/1 | 12 | 0 |
| total | 86 | 81 | +530760/-90148 | 41/17/8/5/15 | 83 | 82 |
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 |
|---|---|---|---|---|
| AG/PSSG - Quartech | 32 | 0 | 0 / 0 / 8 | 11 |
| ap-quartech | 1 | 0 | 0 / 0 / 0 | 0 |
| asanchezr | 11 | 0 | 0 / 0 / 3 | 3 |
| devinleighsmith | 32 | 0 | 0 / 0 / 8 | 11 |
| dfilteau | 5 | 0 | 0 / 0 / 0 | 0 |
| eddherrera | 14 | 2 | 0 / 0 / 8 | 11 |
| FuriousLlama | 9 | 1 | 1 / 0 / 19 | 20 |
| PIMS Developers | 11 | 0 | 0 / 0 / 3 | 3 |
| stairaku | 9 | 0 | 0 / 0 / 6 | 7 |
| total | 81 | 3 | 1 / 0 / 44 | 52 |
The most commented PRs
- psp-8630 | Updated Leases and added renewals(23)
- PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(11)
- PSP-8317: Lease and Licence Edit/View Fee Denomination forms(11)
- PSP-7158 Show markers positioned corresponding to that file (6)
- PSP-8746 | GST is modifiable for periods(4)
Code review engagement total
PR Size - determined using the formula: additions + deletions * 0.5. 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 | Total merged PRs | Agreed / Disagreed / Total discussions conducted | Comments conducted | PR size: xs/s/m/l/xl | Changes requested / Commented / Approved |
|---|---|---|---|---|---|
| AG/PSSG - Quartech | 32 | 1 / 0 / 32 | 36 | 12/5/4/2/8 | 3 / 8 / 26 |
| ap-quartech | 1 | 0 / 0 / 0 | 0 | 0/0/0/0/0 | 0 / 0 / 0 |
| asanchezr | 11 | 0 / 0 / 6 | 8 | 21/9/3/2/4 | 0 / 5 / 38 |
| devinleighsmith | 32 | 1 / 0 / 32 | 36 | 12/5/4/2/8 | 3 / 7 / 31 |
| dfilteau | 5 | 0 / 0 / 0 | 0 | 0/0/0/0/0 | 0 / 0 / 0 |
| eddherrera | 14 | 0 / 0 / 3 | 3 | 10/3/0/1/1 | 0 / 3 / 15 |
| FuriousLlama | 9 | 0 / 0 / 3 | 5 | 4/1/4/0/3 | 0 / 3 / 12 |
| PIMS Developers | 11 | 0 / 0 / 6 | 8 | 21/9/3/2/4 | 0 / 8 / 38 |
| stairaku | 9 | 0 / 0 / 0 | 0 | 0/0/0/0/0 | 0 / 0 / 0 |
| total | 81 | 1 / 0 / 44 | 52 | 41/17/8/5/15 | 3 / 19 / 82 |
Review Request 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 |
|---|---|---|---|---|---|
| AG/PSSG - Quartech | 5 | 26 | 16 hours 53 minutes | 26 hours 18 minutes | 5 hours 22 minutes |
| asanchezr | 7 | 39 | 8 hours 16 minutes | 1 hour 51 minutes | |
| devinleighsmith | 5 | 31 | 16 hours 53 minutes | 26 hours 18 minutes | 5 hours 22 minutes |
| eddherrera | 0 | 15 | 19 hours 27 minutes | ||
| FuriousLlama | 5 | 12 | 17 hours 44 minutes | 53 minutes | |
| PIMS Developers | 7 | 38 | 8 hours 16 minutes | 1 hour 51 minutes | |
| stairaku | 1 | 0 | |||
| total | 18 | 82 | 17 hours 45 minutes | 15 hours 2 minutes | 5 hours 22 minutes |