-
Notifications
You must be signed in to change notification settings - Fork 353
doc: Add AI policy #5879
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
doc: Add AI policy #5879
Conversation
This patch formats the `CONTRIBUTING.md` file, plus it fixes some links, lists, block of code etc.
This patch adds AI policy. For the moment, it's a copy-paste (modulo emphasises) from Forgejo's.
| combination of [git add] -p or git checkout -p to split changes into logical | ||
| units. This makes your work easier to review and reduces the chance of | ||
| introducing unrelated changes. | ||
| combination of [git add] -p or [git checkout] -p to split changes into |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link was missing here.
| that is, just the branch name.) | ||
|
|
||
| # Writing changelog entries | ||
| ## Writing changelog entries |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heading level was invalid here.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5879 +/- ##
==========================================
- Coverage 88.59% 88.59% -0.01%
==========================================
Files 363 363
Lines 102926 102926
Branches 102926 102926
==========================================
- Hits 91192 91186 -6
- Misses 7493 7499 +6
Partials 4241 4241 ☔ View full report in Codecov by Sentry. |
CodSpeed Performance ReportMerging #5879 will not alter performanceComparing Summary
|
manuroe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this addition.
Should we add this change to the changelog?
I would say so at it is quite significant. We could say there that the matrix.org website is doing the same thing (matrix-org/matrix.org#2999).
|
|
||
| ## AI policy | ||
|
|
||
| This policy is a copy of the [Forgejo's AI agreement]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This tag does not match the link provided line 392.
Hmm, changelogs are for users of the SDK while the policy is for developers. I'm not sure users are that interested into the contributing guide or the AI policy. |
This clarification makes sense. |
Does exactly what it says on the tin.