• 11 Posts
  • 186 Comments
Joined 1 year ago
cake
Cake day: June 13th, 2023

help-circle








  • From the PR comments:

    Maintainers MAY merge incorrect patches from other Contributors with the goals of (a) ending fruitless discussions, (b) capturing toxic patches in the historical record, © engaging with the Contributor on improving their patch quality.

    I asked around and asked in the C4 specification matrix room.
    And the reason is actually simple. If you merge bad code, have a record of proof in git (pull requests aren’t forever it’s only a github/gitlab thing).

    So the idea is if you merge bad code you have proof in the git record that there is a bad actor. You can always revert the commit again or fix it. And the record can act as a proof in case the community want to get rid of bad actors.