-
|
Hey there! I see that 2 weeks ago, @jeaye reverted the LLM policy. Good stuff to see and totally reasonable. But looking at the commit message of the change:
The contribution policy seems a bit misleading as it violates the norms of a contribution policy, which is that it applies to all participants. Also, most people will interpret this contribution policy as not allowing LLM generated code within Jank at all. They wouldn't know any better unless they either look at that specific commit or some time in the future see an indication of LLM usage by either co-author or attribution in the commit message of a core Jank developer. But now attribution isn't in the policy at all. Obviously I'm not a contributor/maintainer/core-dev or the creator of Jank, so it's not my place to tell you how to run your project. This is just perspective from a random Lisper taking a look at Jank. Consider or discard. |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment
-
|
The commit message is a footnote. The policy itself states:
Here, "external contributions" are contrasted with "internal contributions". This policy applies to external contributors.
Our code of conduct applies to all contributors, internal or external. It is not uncommon for core maintainers to contribute to a project differently than external contributors. LLVM, for example, has the same practice, where some people are granted commit access and others need approval for merges. That is the same difference between internal and external contributors. |
Beta Was this translation helpful? Give feedback.
The commit message is a footnote. The policy itself states:
Here, "external contributions" are contrasted with "internal contributions". This policy applies to external contributors.
Our code of conduct applies to all contributors, internal or external. It is not uncommon for core maintainers to contribute to a project differently than external contributors. LLVM, for example, has the same practice, where some people are granted commit access and others need approval for merges. …