Pull requests report(19/08/2024 21:02)
- Dominant language
- TSQL
- Stars
- 7
- Forks
- 27
- Avg merge
- 3d 23h
- Merged PRs (30d)
- 24
Description
## Pull Request report total
This report based on 588 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
SHOW_USERS: FuriousLlama,asanchezr,devinleighsmith,eddherrera,stairaku
REPORT_DATE_START: 1/01/2024
REPORT_DATE_END: 31/12/2024
AMOUNT: 100
PERIOD_SPLIT_UNIT: months
EXECUTION_OUTCOME: new-issue
```
## References
- [Pull Request report retrospective timeline](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297449948)
- [Pull Request report 8/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450008)
- [Pull Request report 7/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450086)
- [Pull Request report 6/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450176)
- [Pull Request report 5/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450256)
- [Pull Request report 4/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450328)
- [Pull Request report 3/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450387)
- [Pull Request report 2/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450455)
- [Pull Request report 1/2024](https://github.com/bcgov/PSP/issues/4274#issuecomment-2297450534)
### 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 request | Time to review | Time to approve | Time to merge | Total merged PRs |
| :------: | :------: | :------: | :------: | :------: | :------: |
| **asanchezr** | 21 hours 37 minutes | 16 hours 25 minutes | 16 hours 39 minutes | 18 hours | 94 |
| **devinleighsmith** | 79 hours 28 minutes | 22 hours 8 minutes | 23 hours 35 minutes | 47 hours 4 minutes | 233 |
| **eddherrera** | | 24 hours 50 minutes | 46 hours 25 minutes | 51 hours 26 minutes | 66 |
| **FuriousLlama** | 119 hours 27 minutes | 20 hours 25 minutes | 23 hours 50 minutes | 45 hours 2 minutes | 74 |
| **stairaku** | | 19 hours 8 minutes | 22 hours 24 minutes | 27 hours 27 minutes | 38 |
```mermaid
gantt
title Pull requests timeline(75th percentile) total / minutes
dateFormat X
axisFormat %s
section asanchezr
Time to review request(21 hours 37 minutes) : 0, 1297
Time to review(16 hours 25 minutes) : 0, 985
Time to approve(16 hours 39 minutes) : 0, 999
Time to merge(18 hours) : 0, 1080
section devinleighsmith
Time to review request(79 hours 28 minutes) : 0, 4768
Time to review(22 hours 8 minutes) : 0, 1328
Time to approve(23 hours 35 minutes) : 0, 1415
Time to merge(47 hours 4 minutes) : 0, 2824
section eddherrera
Time to review(24 hours 50 minutes) : 0, 1490
Time to approve(46 hours 25 minutes) : 0, 2785
Time to merge(51 hours 26 minutes) : 0, 3086
section FuriousLlama
Time to review request(119 hours 27 minutes) : 0, 7167
Time to review(20 hours 25 minutes) : 0, 1225
Time to approve(23 hours 50 minutes) : 0, 1430
Time to merge(45 hours 2 minutes) : 0, 2702
section stairaku
Time to review(19 hours 8 minutes) : 0, 1148
Time to approve(22 hours 24 minutes) : 0, 1344
Time to merge(27 hours 27 minutes) : 0, 1647
```
### longest-pending reviews
1. [Dead code/lint update - read description(521 hours 28 minutes)](https://github.com/bcgov/PSP/pull/3713)
2. [SPIKE: Siteminder Refresh(465 hours 21 minutes)](https://github.com/bcgov/PSP/pull/3720)
3. [styled modal that warns the user is leaving a form.(457 hours 19 minutes)](https://github.com/bcgov/PSP/pull/3966)
4. [psp-7986 update ts version - increment packages with vulnerabilities.(315 hours 32 minutes)](https://github.com/bcgov/PSP/pull/3856)
5. [psp-7323 Contact ambiguous(263 hours 26 minutes)](https://github.com/bcgov/PSP/pull/3708)
### longest-pending approvals
1. [SPIKE: Vite (619 hours 32 minutes)](https://github.com/bcgov/PSP/pull/3760)
2. [PSP-8114 : Expose Mayan errors to frontend (temporarily)(484 hours 15 minutes)](https://github.com/bcgov/PSP/pull/3972)
3. [Techdebt/use generated document models(461 hours 36 minutes)](https://github.com/bcgov/PSP/pull/3652)
4. [PSP-7681 : FT-REG: Browser alerts should be replaced by established modal dialogs(317 hours 15 minutes)](https://github.com/bcgov/PSP/pull/3790)
5. [psp-7275 complete a disposition file. psp-7785, psp-7322, psp-7568(259 hours 4 minutes)](https://github.com/bcgov/PSP/pull/3740)
### longest-pending merge
1. [psp-5703: add unit tests for document sync and repository(3884 hours 34 minutes)](https://github.com/bcgov/PSP/pull/3373)
2. [Contact Ambiguous(530 hours 15 minutes)](https://github.com/bcgov/PSP/pull/3666)
3. [Psp 8018 refactor property selector, show frontend warning when selecting isretired properties. (do not review until all 5.3 stories/bugs have been removed).(289 hours 55 minutes)](https://github.com/bcgov/PSP/pull/4052)
4. [Vitest(192 hours 9 minutes)](https://github.com/bcgov/PSP/pull/3950)
5. [psp-7323 Contact ambiguous(174 hours 1 minute)](https://github.com/bcgov/PSP/pull/3708)
### 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 |
| :------: | :------: | :------: | :------: | :------: | :------: | :------: |
| **asanchezr** | 98 | 94 | +1213217/-232109 | 43/12/11/7/25 | 64 | 194 |
| **devinleighsmith** | 253 | 233 | +3545938/-596508 | 137/34/18/7/57 | 184 | 186 |
| **eddherrera** | 73 | 66 | +312348/-75693 | 20/21/12/3/17 | 271 | 104 |
| **FuriousLlama** | 80 | 74 | +248583/-90766 | 22/16/8/12/22 | 228 | 115 |
| **stairaku** | 43 | 38 | +17200/-9806 | 13/5/4/8/13 | 19 | 3 |
### 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 |
| :------: | :------: | :------: | :------: | :------: |
| **asanchezr** | 94 | 0 | 4 / 0 / 34 | 38 |
| **devinleighsmith** | 233 | 3 | 1 / 0 / 100 | 120 |
| **eddherrera** | 66 | 7 | 0 / 0 / 123 | 152 |
| **FuriousLlama** | 74 | 8 | 6 / 0 / 130 | 140 |
| **stairaku** | 38 | 2 | 0 / 0 / 11 | 12 |
### The most commented PRs
1. [PSP-7296 : Create/Edit Sales Details (Disposition)(64)](https://github.com/bcgov/PSP/pull/3731)
2. [PSP-7500 - UI/UX cleanup agreements create /edit(42)](https://github.com/bcgov/PSP/pull/3817)
3. [Tech debt| use new api files and null enhancements(41)](https://github.com/bcgov/PSP/pull/3739)
4. [psp-8630 | Updated Leases and added renewals(23)](https://github.com/bcgov/PSP/pull/4180)
5. [PSP-8315 : Add Lease and License Checklist(23)](https://github.com/bcgov/PSP/pull/4062)
### 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 |
| :------: | :------: | :------: | :------: | :------: | :------: |
| **asanchezr** | 94 | 4 / 0 / 78 | 88 | 87/43/17/15/32 | 0 / 34 / 187 |
| **devinleighsmith** | 233 | 3 / 0 / 205 | 236 | 63/31/22/17/53 | 17 / 57 / 182 |
| **eddherrera** | 66 | 2 / 0 / 32 | 35 | 63/12/6/4/19 | 1 / 15 / 104 |
| **FuriousLlama** | 74 | 2 / 0 / 85 | 107 | 47/15/13/6/34 | 3 / 29 / 106 |
| **stairaku** | 38 | 0 / 0 / 0 | 0 | 0/0/2/1/0 | 0 / 0 / 3 |
### 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 |
| :------: | :------: | :------: | :------: | :------: | :------: |
| **asanchezr** | 38 | 194 | 18 hours 36 minutes | 22 hours 37 minutes | |
| **devinleighsmith** | 52 | 186 | 21 hours 25 minutes | 25 hours | 21 hours 13 minutes |
| **eddherrera** | 18 | 104 | 26 hours | 3 minutes | 1 hour 3 minutes |
| **FuriousLlama** | 32 | 115 | 30 hours 9 minutes | 8 hours 17 minutes | 22 hours 28 minutes |
| **stairaku** | 5 | 3 | 5 hours 38 minutes | 5 hours 37 minutes | |
Contributor guide
No contributing guide indexed for this repository
Research direction
This issue is a generated report from the Pull request analytics action, with monthly statistics and links to related pull requests. Start by reviewing the referenced analytics action and report comments; no source files, tests, or requested change are identified, so completion cannot be defined from this issue alone.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions
- Domain
- analytics
- Issue type
- Documentation
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100