Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 27 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
## Description

<!-- Describe the goal of this PR. What does it change? Why? -->
<!-- Include links to related issues or AgilePlus Work Packages (WPs) if applicable. -->
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The terminology "Work Packages (WPs)" is used here, but CONTRIBUTING.md refers to these as "specs" (e.g., "AgilePlus spec registry") and also mentions "ADRs". Aligning the terminology helps maintain consistency across the project documentation.

Suggested change
<!-- Include links to related issues or AgilePlus Work Packages (WPs) if applicable. -->
<!-- Include links to related issues, AgilePlus specs, or ADRs if applicable. -->


## Type of Change

- [ ] 🐛 Bug fix
- [ ] ✨ New feature
- [ ] ♻️ Refactoring (no functional changes)
- [ ] 📝 Documentation update
- [ ] 🚀 Performance / Infrastructure / CI
- [ ] 🧹 Chore / Dependencies

## Checklist

<!-- Review the following before requesting a review or merging. -->

- [ ] My code follows the project's formatting and style guidelines.
- [ ] I have run local quality checks (`task quality` / `cargo test` / `npm run lint`).
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The checklist mentions task quality and npm run lint, but the CONTRIBUTING.md file indicates this is a polyglot repository using Rust (cargo), Go (go), and Python (uv). The checklist should reflect the actual tools used in the project to avoid confusing contributors.

Suggested change
- [ ] I have run local quality checks (`task quality` / `cargo test` / `npm run lint`).
- [ ] I have run local quality checks (`cargo test`, `go test`, or `uv run pytest`).

- [ ] I have added/updated relevant tests where applicable.
- [ ] I have updated corresponding documentation (if needed).
- [ ] No temporary placeholders (e.g., `TODO`, `FIXME`) are left in the submitted code.

## Additional Notes (Optional)

<!-- Any other context, screenshots, or things reviewers should look out for? -->
Loading