Skip to content

Include AI tool usage disclosure in PR template - #2127

Open
monalehtinen wants to merge 2 commits into
mainfrom
monalehtinen-patch-2
Open

monalehtinen wants to merge 2 commits into
mainfrom
monalehtinen-patch-2

Conversation

@monalehtinen

Copy link
Copy Markdown
Collaborator

Added a section to disclose AI tool usage in PRs. Partially fixes #2124

Reasons for creating this PR

Link to relevant issue(s), if any

  • Closes #

Description of the changes in this PR

Known problems or uncertainties in this PR

Checklist

  • phpUnit tests pass locally with my changes
  • I have added tests that show that the new code works, or tests are not relevant for this PR (e.g. only HTML/CSS changes)
  • The PR doesn't reduce accessibility of the front-end code (e.g. tab focus, scaling to different resolutions, use of .sr-only class, color contrast)
  • The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

Added a section to disclose AI tool usage in PRs. Partially fixes #2124
@codecov

codecov Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.99%. Comparing base (a7d548e) to head (4175bb5).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #2127   +/-   ##
=========================================
  Coverage     70.99%   70.99%           
  Complexity     1723     1723           
=========================================
  Files            34       34           
  Lines          4479     4479           
=========================================
  Hits           3180     3180           
  Misses         1299     1299           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@osma osma left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, but a couple of suggestions:

  • Also add a "How to test" headline. This is very helpful to reviewers especially for PRs generated with AI assistance.
  • Make the minimum level for merging AI:AMBER. Skosmos is a relatively mature project and I think we want to be somewhat careful with AI-authored contributions.

Updated the AI tool usage disclosure, set new minimum level for merging to AI:AMBER. Added heading for testing of PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Define AI tool policy for Skosmos contributions

2 participants