wehub-resource-sync
48 lines
1.1 KiB
Markdown
48 lines
1.1 KiB
Markdown
<!-- Write this PR description in English. -->
|
|
|
|
## What
|
|
|
|
<!-- One or two sentences: what does this PR change? -->
|
|
|
|
## Why
|
|
|
|
<!-- The need this addresses, and who benefits. -->
|
|
|
|
## User-visible impact
|
|
|
|
<!-- Default behavior changes? New config? Breaking changes? "None" is a valid answer. -->
|
|
|
|
---
|
|
|
|
## Self-review
|
|
|
|
Please confirm before requesting review. See [CONTRIBUTING.md](../CONTRIBUTING.md) for details.
|
|
|
|
### 1. Architecture
|
|
|
|
- [ ] Smallest possible diff for the need
|
|
- [ ] No new configuration knobs (or justified below)
|
|
- [ ] No new dependencies (or justified below)
|
|
- [ ] Fits the existing design / naming / layering
|
|
|
|
### 2. Need
|
|
|
|
- [ ] Common need that benefits most users, **or** scope and side effects are explained below
|
|
- [ ] No unintended impact on other users' workflows or defaults
|
|
|
|
### 3. Code
|
|
|
|
- [ ] All tests pass
|
|
- [ ] Coverage does not drop; new code has new tests
|
|
- [ ] Commit messages and this PR are written in English
|
|
|
|
### Bonus
|
|
|
|
- [ ] This PR was authored using OpenClacky itself
|
|
|
|
---
|
|
|
|
## Notes for reviewers
|
|
|
|
<!-- Anything reviewers should focus on, trade-offs you considered, or follow-ups planned. -->
|