Skip to content

Add AI policy to skorch - #1181

Merged
BenjaminBossan merged 1 commit into
masterfrom
ai-add-agent-instructions
Oct 26, 2026
Merged

BenjaminBossan merged 1 commit into
masterfrom
ai-add-agent-instructions

Conversation

@BenjaminBossan

Copy link
Copy Markdown
Collaborator

We're now getting a bunch of (what I assume to be) AI generated PRs in skorch. From our experience, most of those are initiated by asking an AI agent to fix open issues or to check the code for bugs. They are not created because of a real user need and most likely don't help the contributors to learn. As such, I would argue in favor of closing those PRs and not accepting them in the future. This should be reflected in an official AI policy, which this PR provides.

I have already discussed this internally with @githubnemo. @thomasjpfan if you have an opinion on this, please let us know.

We're now getting a bunch of (what I assume to be) AI generated PRs in
skorch. From our experience, most of those are initiated by asking an
AI agent to fix open issues or to check the code for bugs. They are
not created because of a real user need. As such, I would argue in
favor of closing those PRs and not accepting them in the future. This
should be reflected in an official AI policy, which this PR provides.

@githubnemo githubnemo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reads as discussed, LGTM.

@thomasjpfan thomasjpfan 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 reasonable to me

@BenjaminBossan
BenjaminBossan merged commit edf9a7d into master Oct 26, 2026
16 checks passed
@BenjaminBossan
BenjaminBossan deleted the ai-add-agent-instructions branch October 26, 2026 01:42
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