Skip to content

chore: update contribution guidelines#2849

Open
kfiven wants to merge 2 commits intodevfrom
docs-update
Open

chore: update contribution guidelines#2849
kfiven wants to merge 2 commits intodevfrom
docs-update

Conversation

@kfiven
Copy link
Copy Markdown
Collaborator

@kfiven kfiven commented Mar 25, 2026

Description

Fixes #

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 25, 2026

Preview: https://2849--pr-cinny.netlify.app
⚠️ Exercise caution. Use test accounts. ⚠️

help junior developers learn and grow, but if you're interested
in that then don't use AI, and we'll help you.

These rules apply only to outside contributions to Cinny. Maintainers
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Given that there are current copyright concerns about AI generated code, I think it's very important that maintainers at least disclose use of AI

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The essence here is that 'AI as a tool' is okay but 'AI as author' is not, so if being used as tool, such as to understand codebase, understand bug, then we should not prohibit that but if someone is using it to just generate code without understanding anything than that is an issue.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah I agree with that, but I think it's extremely important to document how AI is used. Both for transparency and to dissuade potential future copyright challenges.

- [Matrix JavaScript SDK documentation](https://matrix-org.github.io/matrix-js-sdk/index.html)
> ### Legal Notice
>
> When contributing to this project, you must agree that you have authored 100%
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This line effectively eliminates AI contributions.

Copy link
Copy Markdown
Collaborator Author

@kfiven kfiven Mar 27, 2026

Choose a reason for hiding this comment

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

Well, I think if as above, its being used as tool, then authorship lies on with the user not to AI.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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