Pull requests report(19/08/2024 20:30)
- Dominant language
- TSQL
- Stars
- 7
- Forks
- 27
- Avg merge
- 3d 23h
- Merged PRs (30d)
- 24
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](https://github.com/AlexSim93/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](https://github.com/bcgov/PSP/issues/4273#issuecomment-2297396312)
### 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 |
```mermaid
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
1. [psp-8783 ensure that form state is maintained during collapsing of sidebar.(118 hours 52 minutes)](https://github.com/bcgov/PSP/pull/4168)
2. [psp-8365 lease flexible duration(81 hours 1 minute)](https://github.com/bcgov/PSP/pull/4142)
3. [psp-8708 fix tenant primary contacts.(67 hours 25 minutes)](https://github.com/bcgov/PSP/pull/4176)
4. [PSP-7158 Show markers positioned corresponding to that file (65 hours 44 minutes)](https://github.com/bcgov/PSP/pull/4147)
5. [PSP-8414 : Select one or multiple properties for H120 Generation(65 hours 2 minutes)](https://github.com/bcgov/PSP/pull/4160)
### longest-pending approvals
1. [PSP-8414 : Select one or multiple properties for H120 Generation(26 hours 1 minute)](https://github.com/bcgov/PSP/pull/4160)
2. [psp-8630 | Updated Leases and added renewals(18 hours 46 minutes)](https://github.com/bcgov/PSP/pull/4180)
3. [PSP-8896 : FT:AcquisitionFile:Requisition: System generates warning m…(16 hours 55 minutes)](https://github.com/bcgov/PSP/pull/4205)
4. [PSP-8317: Lease and Licence Edit/View Fee Denomination forms(4 hours 18 minutes)](https://github.com/bcgov/PSP/pull/4187)
5. [PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(2 hours 45 minutes)](https://github.com/bcgov/PSP/pull/4188)
### longest-pending merge
1. [psp-8018: display retired warning immediately upon add.(122 hours 48 minutes)](https://github.com/bcgov/PSP/pull/4164)
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)](https://github.com/bcgov/PSP/pull/4154)
3. [PSP-7158 Show markers positioned corresponding to that file (51 hours 4 minutes)](https://github.com/bcgov/PSP/pull/4147)
4. [psp-8772 lease payment export corrections.(40 hours 52 minutes)](https://github.com/bcgov/PSP/pull/4170)
5. [psp-8545 Correct behaviour when there is no property ownership specif…(21 hours 12 minutes)](https://github.com/bcgov/PSP/pull/4149)
### 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 :+1:.
**Disagreed** - discussions with at least 1 reaction :-1:.
| 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)](https://github.com/bcgov/PSP/pull/4180)
2. [PSP-7980 : Display L/L Status in Property Information Pane - Management (Summary Section)(11)](https://github.com/bcgov/PSP/pull/4188)
3. [PSP-8317: Lease and Licence Edit/View Fee Denomination forms(11)](https://github.com/bcgov/PSP/pull/4187)
4. [PSP-7158 Show markers positioned corresponding to that file (6)](https://github.com/bcgov/PSP/pull/4147)
5. [PSP-8746 | GST is modifiable for periods(4)](https://github.com/bcgov/PSP/pull/4227)
### 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 :+1:.
**Disagreed** - discussions with at least 1 reaction :-1:.
| 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 |
Contributor guide
No contributing guide indexed for this repository
Research direction
This issue is an automated pull-request analytics report for bcgov/psp, generated with the Pull request analytics action. No source files, tests, requested change, or completion criteria are identified, so there is no implementation path to follow; confirm whether a separate maintenance task is intended.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions
- Domain
- analytics
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 1/100