.github/templates: Add info about keeping AI section#22459
Merged
Conversation
AnnsAnns
added
CI: ready for build
Contributor
|
I think the first and second sentences are a bit redundant, but I'll take a closer look when I'm back on the computer. |
Member
Author
they are (intentionally) |
crasbe
reviewed
Jul 10, 2026
| --> | ||
|
|
||
| ### Declaration of AI-Tools / LLMs usage: | ||
|
|
Contributor
There was a problem hiding this comment.
Suggested change
| <!-- | |
| Keep this section as is even if no AI/LLM was used, **do not** remove it if you did not use any AI/LLM. | |
| --> | |
Perhaps put it in front and in a separate comment for more chances of it being read?
Member
Author
There was a problem hiding this comment.
I also wanted to do that originally, just wasn't sure whether that would generate too much noise, will do that tho
crasbe
added
Type: enhancement
kfessel
reviewed
Jul 15, 2026
Comment on lines
+66
to
+67
| Keep this section as is even if no AI/LLM was used, **do not** remove it if you did not use any AI/LLM. | ||
| Thanks for contributing! |
Contributor
There was a problem hiding this comment.
Suggested change
| Keep this section as is even if no AI/LLM was used, **do not** remove it if you did not use any AI/LLM. | |
| Thanks for contributing! | |
| Keep this section even if no AI/LLM was used, **do not** remove it. | |
| --> | |
| <!-- | |
| Thanks for contributing! |
kfessel
reviewed
Jul 15, 2026
| - none | ||
|
|
||
| <!-- | ||
| Keep this section as is even if no AI/LLM was used, **do not** remove it if you did not use any AI/LLM. |
Contributor
There was a problem hiding this comment.
Suggested change
| Keep this section as is even if no AI/LLM was used, **do not** remove it if you did not use any AI/LLM. | |
| Keep this section even if no AI/LLM was used, **do not** remove it. |
Contributor
There was a problem hiding this comment.
its still redundant just a little less
Contributor
Contributor
|
reading #22345 makes me think we should add a general: "keep the structure if not unfit" |
AnnsAnns
force-pushed
the
ai_policy_make_clear_none
branch
from
July 19, 2026 09:06
71d7323 to
e36606e
Compare
Member
Author
|
Squashed 👍 |
crasbe
enabled auto-merge
July 19, 2026 17:19
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contribution description
Sometimes people remove the AI section in the expectation that if none was used the section is useless. Add a smaller reminder to keep it so it hopefully doesn't happen in the future.
Testing procedure
No
Issues/PRs references
Declaration of AI-Tools / LLMs usage:
AI-Tools / LLMs that were used are: