Address closed renovate bot PRs
- Dominant language
- Python
- Stars
- 65
- Forks
- 11
- Avg merge
- 21h 25m
- Merged PRs (30d)
- 70
Description
**Describe the task**
Some PRs have been closed due to the large perceived effort and low priority to update the target dependency. E.g. update node to v16, update devexpress react table used in FWI calculator.
Reopen and address those PRs if still applicable. They are ignored by renovate due to the close PR policy but can be found in the renovate bot logged output: https://github.com/bcgov/wps/runs/5262185415?check_suite_focus=true#step:3:4186
**Acceptance Criteria**
- [ ] Reopen any applicable PR
- [ ] Adjust if necessary
- [ ] Test, deploy, merge
**Additional context**
Renovate lists these PRs as needing to be merged. A lot are closed:
```
DEBUG: Branch lists (repository=bcgov/wps)
"branchList": [
"renovate/actions-checkout-3.x",
"renovate/actions-download-artifact-3.x",
"renovate/actions-setup-python-3.x",
"renovate/actions-upload-artifact-3.x",
"renovate/aiobotocore-2.x",
"renovate/date-io-luxon-2.x",
"renovate/devexpress-dx-react-core-3.x",
"renovate/devexpress-dx-react-grid-3.x",
"renovate/devexpress-dx-react-grid-material-ui-3.x",
~~"renovate/eslint-8.x",~~ // Done in https://github.com/bcgov/wps/pull/1829
"renovate/node-16.x",
"renovate/npm-all-minor-patch",
"renovate/poetry-all-minor-patch",
"renovate/pytest-7.x",
"renovate/pytest_bdd-5.x",
~~"renovate/react-scripts-5.x"~~ // Done in https://github.com/bcgov/wps/pull/1825
],
"renovateBranches": [
"renovate/actions-checkout-3.x",
"renovate/actions-download-artifact-3.x",
"renovate/actions-setup-python-3.x",
"renovate/actions-upload-artifact-3.x",
"renovate/npm-all-minor-patch",
"renovate/pytest-7.x"
]
```
Contributor guide
Research direction
Start with the linked Renovate bot run and compare its logged branch list with the closed pull requests, excluding updates already completed by pull requests 1829 and 1825. Determine which dependency updates remain applicable, then reopen and adjust those PRs as needed. Done means the applicable updates are tested, deployed, and merged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- node.js, python, react
- Domain
- build-system, devops, tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100