Skip to content

chore(pr-template): add AI usage note#21357

Merged
david-allison merged 1 commit into
ankidroid:mainfrom
ericli3690:ericli3690-pr-template-llms
Jul 9, 2026
Merged

chore(pr-template): add AI usage note#21357
david-allison merged 1 commit into
ankidroid:mainfrom
ericli3690:ericli3690-pr-template-llms

Conversation

@ericli3690

@ericli3690 ericli3690 commented Jul 8, 2026

Copy link
Copy Markdown
Member

Purpose / Description

Many PRs now use AI tools. Including this note in PR templates should 1) increase adherence to the AI_POLICY among long-time contributors who may forget to make Assisted-by notes and 2) help inform new contributors about the AI_POLICY.

Fixes

  • Nothing. Just a template change.

Approach

See Purpose/Description above.

How Has This Been Tested?

  • Just a template change. None needed.

Checklist

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code

@ericli3690 ericli3690 self-assigned this Jul 8, 2026

@david-allison david-allison 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.

  • no need for a markdown link as it's a comment
  • consider putting the link on a newline for ease of copyright
  • Remove 'Please' - brevity/respect for time is politeness IMO

I would prefer to put the entire 'NOTE' in a comment: no action needed by default,

Many PRs now use AI tools. Including this note in PR templates should 1) increase adherence to the AI_POLICY among long-time contributors who may forget to make Assisted-by notes and 2) help inform new contributors about the AI_POLICY.
@ericli3690 ericli3690 force-pushed the ericli3690-pr-template-llms branch from 9bf2b89 to a409c46 Compare July 8, 2026 21:44

@criticalAY criticalAY left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cool!

@criticalAY criticalAY added Needs Second Approval Has one approval, one more approval to merge and removed Needs Review labels Jul 8, 2026
@criticalAY criticalAY requested a review from david-allison July 8, 2026 22:06

@david-allison david-allison 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.

BEAUTY

@david-allison david-allison enabled auto-merge July 8, 2026 22:07
@criticalAY criticalAY added Pending Merge Things with approval that are waiting future merge (e.g. targets a future release, CI wait, etc) and removed Needs Second Approval Has one approval, one more approval to merge labels Jul 8, 2026
@david-allison david-allison added this pull request to the merge queue Jul 8, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 8, 2026
@ericli3690 ericli3690 added this pull request to the merge queue Jul 8, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 8, 2026
@ericli3690

Copy link
Copy Markdown
Member Author

@david-allison Is something up with code-cov? Only the ubuntu check is failing in the merge queue, but not due to an actual test failure. Happened twice:

==> Verifying GPG signature integrity
 -> Downloading https://cli.codecov.io/latest/linux/codecov.SHA256SUM
 -> Downloading https://cli.codecov.io/latest/linux/codecov.SHA256SUM.sig
 
gpg: Signature made Tue Apr 21 19:28:03 2026 UTC
gpg:                using RSA key 27034E7FDB850E0BBC2C62FF806BB28AED779869
gpg: BAD signature from "Codecov Uploader (Codecov Uploader Verification Key) <security@codecov.io>" [unknown]
==> Could not verify signature. Please contact Codecov if problem continues
    Exiting...
Error: Process completed with exit code 1.

@david-allison

Copy link
Copy Markdown
Member

@ericli3690 check the repo for issues. It flakes far more than it should

@david-allison david-allison added this pull request to the merge queue Jul 8, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 8, 2026
@david-allison david-allison added this pull request to the merge queue Jul 8, 2026
Merged via the queue into ankidroid:main with commit 43a83d1 Jul 9, 2026
21 of 22 checks passed
@github-actions github-actions Bot added this to the 2.25 release milestone Jul 9, 2026
@github-actions github-actions Bot removed the Pending Merge Things with approval that are waiting future merge (e.g. targets a future release, CI wait, etc) label Jul 9, 2026
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.

3 participants