Skip to content

In the contribution guidelines, remove the Generated-By label - #334

Merged
Gerrit91 merged 1 commit into
metal-stack:mainfrom
GeertJohan:generated-or-assisted
Oct 20, 2026
Merged

Gerrit91 merged 1 commit into
metal-stack:mainfrom
GeertJohan:generated-or-assisted

Conversation

@GeertJohan

@GeertJohan GeertJohan commented Jul 30, 2026 •

Copy link
Copy Markdown
Contributor

This may be a little bit nit-picky. I think "Generated-By' should be reserved for commits that are purely generated by tooling (e.g. dependabot?). Also I belive that using "Assisted" conveys better that the final commit is still the responsibility of the person who created it.
https://docs.kernel.org/process/coding-assistants.html

@GeertJohan
GeertJohan requested a review from a team as a code owner July 30, 2026 12:40
@metal-robot metal-robot Bot added the area: documentation Affects the documentation area. label Jul 30, 2026
@netlify

netlify Bot commented Jul 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for metal-stack-io ready!

Name Link
🔨 Latest commit 9ff4c61
🔍 Latest deploy log https://app.netlify.com/projects/metal-stack-io/deploys/6ad6f612c90776e8c981b92c
😎 Deploy Preview https://deploy-preview-334--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
Copy Markdown
Contributor

Assisted-by is definitely better, but I never saw anyone really adding this into the commit messages (me included). What is used consistently is the PR template for indication. So we either have to be more strict or we remove this paragraph from the contribution guideline. I think I would vote for option two if possible.

@metal-stack/website-maintainers What is your take on this?

@vknabel

vknabel commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

@Gerrit91 I agree on removing this paragraph

@Gerrit91 Gerrit91 added the triage This should be talked about in the next planning. label Aug 17, 2026
@vknabel vknabel moved this to In Progress in Development Aug 24, 2026
@metal-robot metal-robot Bot removed the triage This should be talked about in the next planning. label Aug 24, 2026
@vknabel

vknabel commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

@GeertJohan thanks for you patience! Could you remove the Assisted-By label altogether? We decided the PR description should be sufficient. :)

@GeertJohan
GeertJohan force-pushed the generated-or-assisted branch from f191c3c to 9ff4c61 Compare October 20, 2026 05:03
@GeertJohan GeertJohan changed the title In the contribution guidelines, change the Generated-By label to Assisted-By In the contribution guidelines, remove the Generated-By label Oct 20, 2026
@GeertJohan

Copy link
Copy Markdown
Contributor Author

@vknabel sorry for the delay, done now :)

@vknabel vknabel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No worries :)

@Gerrit91
Gerrit91 merged commit 2178a15 into metal-stack:main Oct 20, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Development Oct 20, 2026
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: Done

Development

Successfully merging this pull request may close these issues.

5 participants