-
Notifications
You must be signed in to change notification settings - Fork 0
Create AI_POLICY.md #1
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.
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,11 @@ | ||||||
| Thank you for your interest in contributing to or supporting Geyser! When working with AI tools, here are the parameters we ask that you follow: | ||||||
| - Please refrain from using LLMs in any discussion on GitHub. Specifically for submitting PRs, we want to hear the changes with your own words, and you should be able to understand what a PR is doing without consulting an LLM. | ||||||
| - Using a translator is excluded from these rules, but please also add your original language text under English. | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| - When submitting PRs, please be transparent with any AI tool used to assist with your code, along with what extent the tool was used for. This can just be one sentence: "This PR used \[specific AI tool\] to generate Javadocs." | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. "alongside to which" I guess this is correct but seems a bit hard to parse. Maybe "and include to"? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
would make more sense to me |
||||||
| - Pull requests must always be tested by a human before opening, regardless of AI usage. | ||||||
| - Please keep PRs focused on the relevant change or changes. Excessive whitespacing changes or changes outside the scope of the PR are not allowed. | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Doesn't seem related to AI, so we could probably just remove this line or add it to existing contributor guidelines?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. AI-coauthored PRs do occasionally introduce unrelated changes or "optimizations", but I agree that with the way it's currently written in, it should be moved elsewhere |
||||||
| - While usage of media assets is rare in our projects, please refrain from using AI-generated media for these use-cases - images, video, audio, etc. We prefer bad human drawings over "good" AI creations! | ||||||
|
|
||||||
| Excessive use of misused AI will result in closed PRs or bans from contributing to Geyser. We understand that the codebase can be daunting to newcomers, and for those genuinely interested in contributing we are more than happy to assist and teach proper usage! | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
|
||||||
| Thank you! | ||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.