Skip to content
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

Fix: Contribution guidelines #2049

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Fix: Contribution guidelines #2049

merged 1 commit into from
Jan 9, 2025

Conversation

dragonmux
Copy link
Member

Detailed description

In this PR we have updated the contribution guidelines to set forth the project policy on LLM/"AI" generated contributions.

There are 3 reasons for doing this:

  • We have determined that there is an unacceptable risk from the training data used for these models that copyrighted work may end up in a contribution without proper attribution or licensing, and that the license of the source work may not even be compatible with the project's
  • It places an undue burden on the maintainers to have to work through and spot subtle mistakes that even the contributor doesn't understand as a result, to work out what the correct behaviour should be
  • There is a growing body of research showing these tools hurt productivity instead of help, while increasing negative outcomes such as poor code quality and increased incidence of subtle bugs

All in all, we would rather just distance ourselves from these problems and be able to put the energy they'd waste into making the project as good as we can instead. Fixing bugs, improving target support, implementing new architectures, and making it the best debug server we can.

Your checklist for this pull request

Closing issues

@dragonmux dragonmux requested a review from esden January 9, 2025 21:31
@dragonmux dragonmux added Bug Confirmed bug Documentation Project documentation labels Jan 9, 2025
@dragonmux dragonmux added this to the v2.0 release milestone Jan 9, 2025
Copy link
Member

@esden esden left a comment

Choose a reason for hiding this comment

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

Thank you for putting this rule addition together. We have now had extensive conversation about this issue on discord. I am in 💯 agreement with this change.

@esden esden merged commit 1559132 into main Jan 9, 2025
36 checks passed
@dragonmux dragonmux deleted the fix/contrib-guidelines branch January 9, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Confirmed bug Documentation Project documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants