Pull requests report(19/08/2024 20:30)

[devinleighsmith]

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 Request report 7/2024

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
        

Loading

longest-pending reviews

1. psp-8783 ensure that form state is maintained during collapsing of sidebar.(118 hours 52 minutes)
2. psp-8365 lease flexible duration(81 hours 1 minute)
3. psp-8708 fix tenant primary contacts.(67 hours 25 minutes)
4. PSP-7158 Show markers positioned corresponding to that file (65 hours 44 minutes)
5. PSP-8414 : Select one or multiple properties for H120 Generation(65 hours 2 minutes)

longest-pending approvals

1. PSP-8414 : Select one or multiple properties for H120 Generation(26 hours 1 minute)
2. psp-8630 | Updated Leases and added renewals(18 hours 46 minutes)
3. PSP-8896 : FT:AcquisitionFile:Requisition: System generates warning m…(16 hours 55 minutes)
4. PSP-8317: Lease and Licence Edit/View Fee Denomination forms(4 hours 18 minutes)
5. PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(2 hours 45 minutes)

longest-pending merge

1. psp-8018: display retired warning immediately upon add.(122 hours 48 minutes)
2. PSP-8708: Organization Tenants with person contacts are not displayed when tenant is selected. Small change on Contacts List View(75 hours 46 minutes)
3. PSP-7158 Show markers positioned corresponding to that file (51 hours 4 minutes)
4. psp-8772 lease payment export corrections.(40 hours 52 minutes)
5. 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

1. psp-8630 | Updated Leases and added renewals(23)
2. PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(11)
3. PSP-8317: Lease and Licence Edit/View Fee Denomination forms(11)
4. PSP-7158 Show markers positioned corresponding to that file (6)
5. 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

[devinleighsmith]

Pull Request report 7/2024

References

• Pull Request report total

Pull requests timeline(75th percentile) 7/2024

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) 7/2024 / 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
        

Loading

longest-pending reviews

1. psp-8783 ensure that form state is maintained during collapsing of sidebar.(118 hours 52 minutes)
2. psp-8365 lease flexible duration(81 hours 1 minute)
3. psp-8708 fix tenant primary contacts.(67 hours 25 minutes)
4. PSP-7158 Show markers positioned corresponding to that file (65 hours 44 minutes)
5. PSP-8414 : Select one or multiple properties for H120 Generation(65 hours 2 minutes)

longest-pending approvals

1. PSP-8414 : Select one or multiple properties for H120 Generation(26 hours 1 minute)
2. psp-8630 | Updated Leases and added renewals(18 hours 46 minutes)
3. PSP-8896 : FT:AcquisitionFile:Requisition: System generates warning m…(16 hours 55 minutes)
4. PSP-8317: Lease and Licence Edit/View Fee Denomination forms(4 hours 18 minutes)
5. PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(2 hours 45 minutes)

longest-pending merge

1. psp-8018: display retired warning immediately upon add.(122 hours 48 minutes)
2. PSP-8708: Organization Tenants with person contacts are not displayed when tenant is selected. Small change on Contacts List View(75 hours 46 minutes)
3. PSP-7158 Show markers positioned corresponding to that file (51 hours 4 minutes)
4. psp-8772 lease payment export corrections.(40 hours 52 minutes)
5. psp-8545 Correct behaviour when there is no property ownership specif…(21 hours 12 minutes)

Contribution stats 7/2024

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 7/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
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

1. psp-8630 | Updated Leases and added renewals(23)
2. PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(11)
3. PSP-8317: Lease and Licence Edit/View Fee Denomination forms(11)
4. PSP-7158 Show markers positioned corresponding to that file (6)
5. PSP-8746 | GST is modifiable for periods(4)

Code review engagement 7/2024

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) 7/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
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