Skip to content

[chore] Update PR template - add AI usage disclosure #3596

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
lmolkova wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[chore] Update PR template - add AI usage disclosure #3596

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
16 changes: 11 additions & 5 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,18 @@ Fixes #
Please provide a brief description of the changes here.

> [!IMPORTANT]
> Pull requests acceptance are subject to the triage process as described in [Issue and PR Triage Management](https://github.com/open-telemetry/semantic-conventions/blob/main/issue-management.md).
> PRs that do not follow the guidance above, may be automatically rejected and closed.
> Pull request acceptance is subject to the triage process as described in [Issue and PR Triage Management](https://github.com/open-telemetry/semantic-conventions/blob/main/issue-management.md).
> PRs that do not follow the guidance above may be automatically rejected and closed.

## Merge requirement checklist

* [ ] [CONTRIBUTING.md](https://github.com/open-telemetry/semantic-conventions/blob/main/CONTRIBUTING.md) guidelines followed.
* [ ] Change log entry added, according to the guidelines in [When to add a changelog entry](https://github.com/open-telemetry/semantic-conventions/blob/main/CONTRIBUTING.md#when-to-add-a-changelog-entry).
Copy link
Copy Markdown
Member Author

@lmolkova lmolkova Apr 4, 2026

Choose a reason for hiding this comment

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

we have a CI check for it, no need?

Copy link
Copy Markdown
Member

@arminru arminru Apr 7, 2026

Choose a reason for hiding this comment

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

Either way is fine but this way the author gets a hint even before opening the PR, potentially reducing noise by the failed check + update.

* If your PR does not need a change log, start the PR title with `[chore]`
* [ ] Links to the prototypes or existing instrumentations (when adding or changing conventions)
* [ ] Links to prototypes or existing instrumentations (when adding or changing conventions)
* [ ] Disclose AI usage, see [OTel GenAI policy](https://github.com/open-telemetry/community/blob/main/policies/genai.md):
* [ ] no AI used
* [ ] AI-assisted
* [ ] bulk AI-generated
Comment on lines +17 to +18
Copy link
Copy Markdown
Member

@arminru arminru Apr 7, 2026

Choose a reason for hiding this comment

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

Suggested change
* [ ] AI-assisted
* [ ] bulk AI-generated
* [ ] AI-assisted but mostly my own work
* [ ] bulk AI-generated but I reviewed it in full before submitting
* [ ] bulk AI-generated and not fully reviewed yet

Perhaps? If we believe PR authors would be honest enough to indicate they haven't read what they're submitting.

* [ ] I have the experience and knowledge necessary to understand, review, and validate all content in this PR.[^I-know-my-stuff]
Copy link
Copy Markdown

@horovits horovits Apr 6, 2026

Choose a reason for hiding this comment

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

worth spelling out the ownership, as in owning the PR, as well? (following up on today's SIG call discussion)


[^I-know-my-stuff]:
Yes, I can answer maintainer questions about the content of this PR without using AI.
Loading