mirror of
https://github.com/liyunfan1223/mod-playerbots.git
synced 2026-06-20 15:39:25 +02:00
PR template fix (#2216)
Maintenance PR to remove tasklist visible on github
This commit is contained in:
parent
d6f396ab50
commit
545b21ec0c
@ -43,21 +43,21 @@ any impact on performance, you may skip these question. If necessary, a maintain
|
|||||||
## Impact Assessment
|
## Impact Assessment
|
||||||
<!-- As a generic test, before and after measure of pmon (playerbot pmon tick) can help you here. -->
|
<!-- As a generic test, before and after measure of pmon (playerbot pmon tick) can help you here. -->
|
||||||
- Does this change increase per-bot/per-tick processing or risk scaling poorly with thousands of bots?
|
- Does this change increase per-bot/per-tick processing or risk scaling poorly with thousands of bots?
|
||||||
- [ ] No, not at all
|
- - [ ] No, not at all
|
||||||
- [ ] Minimal impact (**explain below**)
|
- - [ ] Minimal impact (**explain below**)
|
||||||
- [ ] Moderate impact (**explain below**)
|
- - [ ] Moderate impact (**explain below**)
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
- Does this change modify default bot behavior?
|
- Does this change modify default bot behavior?
|
||||||
- [ ] No
|
- - [ ] No
|
||||||
- [ ] Yes (**explain why**)
|
- - [ ] Yes (**explain why**)
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
- Does this change add new decision branches or increase maintenance complexity?
|
- Does this change add new decision branches or increase maintenance complexity?
|
||||||
- [ ] No
|
- - [ ] No
|
||||||
- [ ] Yes (**explain below**)
|
- - [ ] Yes (**explain below**)
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
@ -68,8 +68,8 @@ the message is in a translatable format, and list in the table the message_key a
|
|||||||
Search for GetBotTextOrDefault in the codebase for examples.
|
Search for GetBotTextOrDefault in the codebase for examples.
|
||||||
-->
|
-->
|
||||||
Does this change add bot messages to translate?
|
Does this change add bot messages to translate?
|
||||||
- [ ] No
|
- - [ ] No
|
||||||
- [ ] Yes (**list messages in the table**)
|
- - [ ] Yes (**list messages in the table**)
|
||||||
|
|
||||||
| Message key | Default message |
|
| Message key | Default message |
|
||||||
| --------------- | ------------------ |
|
| --------------- | ------------------ |
|
||||||
@ -82,8 +82,8 @@ AI assistance is allowed, but all submitted code must be fully understood, revie
|
|||||||
We expect contributors to be honest about what they do and do not understand.
|
We expect contributors to be honest about what they do and do not understand.
|
||||||
-->
|
-->
|
||||||
Was AI assistance used while working on this change?
|
Was AI assistance used while working on this change?
|
||||||
- [ ] No
|
- - [ ] No
|
||||||
- [ ] Yes (**explain below**)
|
- - [ ] Yes (**explain below**)
|
||||||
<!--
|
<!--
|
||||||
If yes, please specify:
|
If yes, please specify:
|
||||||
- Purpose of usage (e.g. brainstorming, refactoring, documentation, code generation).
|
- Purpose of usage (e.g. brainstorming, refactoring, documentation, code generation).
|
||||||
@ -94,10 +94,10 @@ If yes, please specify:
|
|||||||
|
|
||||||
## Final Checklist
|
## Final Checklist
|
||||||
|
|
||||||
- [ ] Stability is not compromised.
|
- - [ ] Stability is not compromised.
|
||||||
- [ ] Performance impact is understood, tested, and acceptable.
|
- - [ ] Performance impact is understood, tested, and acceptable.
|
||||||
- [ ] Added logic complexity is justified and explained.
|
- - [ ] Added logic complexity is justified and explained.
|
||||||
- [ ] Documentation updated if needed (Conf comments, WiKi commands).
|
- - [ ] Documentation updated if needed (Conf comments, WiKi commands).
|
||||||
|
|
||||||
## Notes for Reviewers
|
## Notes for Reviewers
|
||||||
<!-- Anything else that's helpful to review or test your pull request. -->
|
<!-- Anything else that's helpful to review or test your pull request. -->
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user