Conversation
MaxGraey
commented
Apr 10, 2026
MaxGraey
commented
- I've read the contributing guidelines
- I've added my name and email to the NOTICE file
|
Perhaps just so it's less overwhelming? |
|
How about to make all these details optional? This is more of a template for a quick description than a mandatory form. |
| - [ ] AI was used for `code` | ||
| - [ ] AI was used for `tests` | ||
| - [ ] AI was used for `documentation` | ||
| - [ ] AI was used for `PR` or `commit` text |
There was a problem hiding this comment.
I think(?) preferably PRs/commit messages shouldnt contain LLM-generated text, unless it's concise. LLMs seem to have a habit of making things kinda long (by default?)...
There was a problem hiding this comment.
I think for commit msg is ok for PR's comment description... yeah, I'm not fan of verbose descriptions which usually generates by LLMs
There was a problem hiding this comment.
For commit I think it is ok since I never read commit message and we will drop it during merging.
For PR title, AI do much better then non english native contributor to be honest.
For PR description, I agree it. 😆
4 participants
