Skip to content

commit: add assisted-by trailer#784

Merged
figsoda merged 1 commit into
mainfrom
push-xksxqyvoutkv
May 3, 2026
Merged

commit: add assisted-by trailer#784
figsoda merged 1 commit into
mainfrom
push-xksxqyvoutkv

Conversation

@figsoda
Copy link
Copy Markdown
Member

@figsoda figsoda commented Apr 30, 2026

@pbsds
Copy link
Copy Markdown

pbsds commented May 2, 2026

The proposed Assisted-by: requirement technically only applies to LLM tools, meaning it is possible to reword this commit trailer. Even so I believe having such a disclosure is both ethical and a net benefit as it gives the nixpkgs reviewer insight into how the derivation was authored, and sets and example about transparency to other contributors. 👍

@figsoda figsoda marked this pull request as ready for review May 3, 2026 20:23
@figsoda
Copy link
Copy Markdown
Member Author

figsoda commented May 3, 2026

it seems that even though this is not required, people seem to find it helpful

I'll go ahead and merge this then

@figsoda figsoda merged commit 684b5c8 into main May 3, 2026
3 checks passed
@figsoda figsoda deleted the push-xksxqyvoutkv branch May 3, 2026 20:25
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.

2 participants