Skip to content

Add AI tools paragraph to contribution guideline.#204

Open
Gerrit91 wants to merge 3 commits intomainfrom
ai-chapter
Open

Add AI tools paragraph to contribution guideline.#204
Gerrit91 wants to merge 3 commits intomainfrom
ai-chapter

Conversation

@Gerrit91
Copy link
Contributor

Description

Add an AI chapter on how we want to proceed with AI generated contributions.

@metal-robot metal-robot bot added the area: documentation Affects the documentation area. label Feb 19, 2026
@netlify
Copy link

netlify bot commented Feb 19, 2026

Deploy Preview for metal-stack-io ready!

Name Link
🔨 Latest commit b4ac8ee
🔍 Latest deploy log https://app.netlify.com/projects/metal-stack-io/deploys/699d8962238ca10008058309
😎 Deploy Preview https://deploy-preview-204--metal-stack-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Gerrit91
Copy link
Contributor Author

For now I lean towards a permissive policy regarding the contributions despite the unclear situation regarding copyright and licensing. Let me know your view about it.

@majst01
Copy link
Contributor

majst01 commented Feb 19, 2026

My thought / possible additions:

  • Add automatic PR size detection into t-shirt sizes, depending on LOC and complexity and tell the PR author that large PRs which are AI assisted can be closed because of too large
  • Let the author acknowledge that there is no license violation of other parties in the contributed code
  • We must clarify if it is now time to make a CLA bot mandatory

Maybe more from my side later.

@Gerrit91
Copy link
Contributor Author

For the metal-robot the CLA bot functionality is probably too complex, this would require us to search an external tool that can do it. T-shirt sizing maybe works depending on how difficult we want to measure code complexity.

@Gerrit91 Gerrit91 added the triage This should be talked about in the next planning. label Feb 19, 2026
@majst01
Copy link
Contributor

majst01 commented Feb 20, 2026

For the metal-robot the CLA bot functionality is probably too complex, this would require us to search an external tool that can do it. T-shirt sizing maybe works depending on how difficult we want to measure code complexity.

This should not be done by the metal-robot, there are plenty of cla bots out there:

https://github.com/marketplace/cla-bot-cla-automation
https://easycla.lfx.linuxfoundation.org/

As we are Linux Foundation Member, we should probably use:
https://github.com/apps/linux-foundation-easycla

and others, we should together decide which to use

@Gerrit91 Gerrit91 moved this to In Progress in Development Feb 23, 2026
@metal-robot metal-robot bot removed the triage This should be talked about in the next planning. label Feb 23, 2026
@Gerrit91 Gerrit91 marked this pull request as ready for review March 9, 2026 13:18
@Gerrit91 Gerrit91 requested a review from a team as a code owner March 9, 2026 13:18
Copy link

@mwindower mwindower left a comment

Choose a reason for hiding this comment

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

ack also from @nikrip

@majst01
Copy link
Contributor

majst01 commented Mar 9, 2026

Still struggle how we could assure that the PR author does not violate the licenses without the help of a CLA Robot

- Communication: Comments, motivations and issues are genuinely written by humans.
- Transparency: Commits where substantial parts were generated by AI must include:
- Commit labels for the review process, like: `Generated-By: [tool name] <your commit message>` .
- The names of the tools that were used with AI should be included in the pull request description. For this, you can find a snippet in the pull request template. As we usually squash commits when we merge, the commit labeling described above is not sufficient.
Copy link
Contributor

@iljarotar iljarotar Mar 12, 2026

Choose a reason for hiding this comment

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

Maybe also add it to the PR title like:

"[AI assisted] your PR title"

Then we can see those commits directly in the history.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: documentation Affects the documentation area.

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants