Skip to content
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

update CONTRIBUTING.md with DCO #502

Merged
merged 2 commits into from
Jul 25, 2024
Merged

update CONTRIBUTING.md with DCO #502

merged 2 commits into from
Jul 25, 2024

Conversation

joshsimmons
Copy link
Member

The Matrix.org Foundation is rolling out this DCO across all public repositories in its namespace. Of note, this DCO does not require the use of "real" or "legally identifiable" names.

@joshsimmons joshsimmons requested a review from a team as a code owner July 24, 2024 17:26
@@ -125,6 +125,59 @@ When reviewing a PR, a maintainer will:
they have acknowledged the message and will make the change.
- Approve a PR which is ready to merge, or nearly ready with some minor tweaks or optional improvements.

### Sign off
Copy link
Contributor

Choose a reason for hiding this comment

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

N.B. just to mention that we link to the Synapse sign off page above. You might wanna adjust that link. https://github.com/matrix-org/matrix-appservice-bridge/pull/502/files#diff-eca12c0a30e25b4b46522ebf89465a03ba72a03f540796c979137931d8f92055R94

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh yes good catch, this is as good a time as any to update that link. Thank you!

Copy link
Contributor

@Half-Shot Half-Shot left a comment

Choose a reason for hiding this comment

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

Good modulo the link

@joshsimmons joshsimmons merged commit 20aed10 into develop Jul 25, 2024
7 of 8 checks passed
@joshsimmons joshsimmons deleted the update-dco branch July 25, 2024 21:03
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