Skip to content
Open
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
29 changes: 14 additions & 15 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,20 +1,19 @@
## Summary

<!-- What does this PR do, in 1-3 sentences -->
# Pull Request Description

## Changes
## Summary
<!-- What does this PR do? -->

<!-- Bullet list of key changes -->
-
## Type of Change
- [ ] Bug fix
- [ ] New feature
- [ ] Breaking change
- [ ] Documentation update

## Testing
<!-- How was this tested? -->

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WARNING: Removed specific testing checkboxes from Testing section

The old template included explicit prompts for local lint, local tests, and manual smoke test. These were replaced with only a generic comment. Contributors no longer get guided reminders to verify lint, run tests, or perform manual smoke tests before requesting review.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.


<!-- How was this verified? -->
- [ ] Local lint passes
- [ ] Local tests pass
- [ ] Manual smoke test (if applicable)

## Related

<!-- Issues/PRs/specs this addresses -->
Closes #
## Checklist

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WARNING: Removed ## Related section with Closes # issue-linking format

Contributors lose the standard GitHub prompt to specify issues this PR closes. Without this, issues may not auto-close on merge, and related work may go unlinked.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

- [ ] Code follows project style guidelines
- [ ] Self-review completed
- [ ] Tests added/updated
- [ ] Documentation updated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SUGGESTION: Redundant "Documentation updated" checkbox in Checklist

"Documentation updated" (line 19) duplicates "Documentation update" in the Type of Change section. Consider consolidating to avoid redundant checkboxes that could confuse contributors.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

Loading