Skip to content

Conversation

@Hywan
Copy link
Member

@Hywan Hywan commented Nov 18, 2025

Does exactly what it says on the tin.

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.
@Hywan Hywan requested a review from a team as a code owner November 18, 2025 17:36
@Hywan Hywan requested review from poljar and stefanceriu and removed request for a team and stefanceriu November 18, 2025 17:36
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
Copy link
Member Author

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
Copy link
Member Author

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.

@Hywan Hywan requested a review from manuroe November 18, 2025 17:39
@codecov
Copy link

codecov bot commented Nov 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.59%. Comparing base (d860749) to head (f2fb08e).
✅ All tests successful. No failed tests found.

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.
📢 Have feedback on the report? Share it here.

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 18, 2025

CodSpeed Performance Report

Merging #5879 will not alter performance

Comparing Hywan:doc-ai-policy (f2fb08e) with main (d860749)

Summary

✅ 50 untouched

Copy link
Contributor

@manuroe manuroe left a 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].
Copy link
Contributor

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.

@poljar
Copy link
Contributor

poljar commented Nov 19, 2025

Should we add this change to the changelog?

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.

@manuroe
Copy link
Contributor

manuroe commented Nov 19, 2025

Should we add this change to the changelog?

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.

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.

3 participants