close
Skip to content

Enforce new AI Disclosure in PR Checklist#1012

Merged
arminsabouri merged 1 commit into
payjoin:masterfrom
DanGould:ai-disclosure
Aug 28, 2025
Merged

Enforce new AI Disclosure in PR Checklist#1012
arminsabouri merged 1 commit into
payjoin:masterfrom
DanGould:ai-disclosure

Conversation

@DanGould
Copy link
Copy Markdown
Contributor

@DanGould DanGould commented Aug 28, 2025

Checking 'a human read this' has proven insufficient. Let's try another experiment where we ask for disclosure. Perhaps this can help us learn more effective AI development techniques at the same time by sharing what we've done.

Inspired by Ghostty: ghostty-org/ghostty#8289
Close #990

Pull Request Checklist

Please confirm the following before requesting review:

  • A human has reviewed every single line of this code before opening the PR (no auto-generated, unreviewed LLM/robot submissions).
  • I have read CONTRIBUTING.md and rebased my branch to produce hygienic commits.

Checking 'a human read this' has proven insufficient. Let's try another
experiment where we ask for disclosure. Perhaps this can help us learn more
effective AI development techniques at the same time by sharing what we've
done.

Inspired by Ghostty: <ghostty-org/ghostty#8289>
Close payjoin#990
@DanGould DanGould requested a review from spacebear21 August 28, 2025 15:41
@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 17300854314

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 85.775%

Totals Coverage Status
Change from base Build 17299929671: 0.0%
Covered Lines: 8014
Relevant Lines: 9343

💛 - Coveralls

@DanGould DanGould requested review from benalleng and removed request for spacebear21 August 28, 2025 15:47
Copy link
Copy Markdown
Contributor

@arminsabouri arminsabouri left a comment

Choose a reason for hiding this comment

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

Ack 67bf7d7

@arminsabouri arminsabouri merged commit ad35923 into payjoin:master Aug 28, 2025
10 checks passed
@benalleng
Copy link
Copy Markdown
Collaborator

Ack 67bf7d7 I know it can be tedious but I think we should try to lead the charge trying to be thorough with these disclosures, I think it could also be helpful for new contributors to see both
A. that we take these disclosures seriously
and
B. give devs a way to share some knowledge about how we are using AI in our everyday workflow

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.

Add Disclosure on use of AI-generated content in contributions

4 participants