Skip to content

Commit 42072c7

Browse files
authored
Revise PR template for clarity and AI code usage
Updated the pull request template to clarify the checklist items and AI-generated code usage.
1 parent fb68103 commit 42072c7

1 file changed

Lines changed: 9 additions & 4 deletions

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,20 @@
1-
<!-- This is just a checklist to guide you. Please remove it if you check all items. -->
1+
<!-- This is a checklist of actions required to have a PR reviewed. You should include it and fill it accordingly. (You may only remove it if you check all items and you are a well-known contributor.) -->
22

3-
**Checklist:**
3+
**Checklist :**
44

55
- [ ] If you are new to Scapy: I have checked [CONTRIBUTING.md](https://github.com/secdev/scapy/blob/master/CONTRIBUTING.md) (esp. section submitting-pull-requests)
66
- [ ] I squashed commits belonging together
77
- [ ] I added unit tests or explained why they are not relevant
88
- [ ] I executed the regression tests (using `tox`)
99
- [ ] If the PR is still not finished, please create a [Draft Pull Request](https://github.blog/2019-02-14-introducing-draft-pull-requests/)
10-
- [ ] This PR uses (partially) AI-generated code: Yes/No
10+
- [ ] This PR uses (partially) AI-generated code. If so:
11+
- [ ] I ensured the generated code follows the internal concepts of scapy
12+
- [ ] This PR has a test coverage > 90%
13+
- [ ] I reviewed every generated line
14+
- [ ] If this PR contains more than 500 lines of code (excluding unit tests) I considered splitting this PR.
15+
- [ ] I considered interoperability tests with existing packages or utilities to ensure conformity of a newly generated protocol
1116

12-
**Failing to mention the use of AI may result in a ban. We do not forbid it, but you must play fair. Be warned !**
17+
**I understand that failing to mention the use of AI may result in a ban. (We do not forbid it, but you must play fair. Be warned !)**
1318

1419
<!-- brief description what this PR will do, e.g. fixes broken dissection of XXX -->
1520

0 commit comments

Comments
 (0)