Skip to content

Check consistency of group info - #4670

Merged
pcapriotti merged 14 commits into
developfrom
group-info-check
Jul 24, 2025
Merged

Check consistency of group info#4670
pcapriotti merged 14 commits into
developfrom
group-info-check

Conversation

@pcapriotti

@pcapriotti pcapriotti commented Jul 17, 2025

Copy link
Copy Markdown
Contributor

This PR adds an optional consistency check of the submitted group info on any commit. The ratchet tree extension is parsed and compared with the index map of the group after the commit is applied.

https://wearezeta.atlassian.net/browse/WPB-18394

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jul 21, 2025
@CLAassistant

CLAassistant commented Jul 21, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@pcapriotti
pcapriotti marked this pull request as ready for review July 21, 2025 09:24
@pcapriotti
pcapriotti requested review from a team as code owners July 21, 2025 09:24
@eyeinsky
eyeinsky force-pushed the group-info-check branch 2 times, most recently from e3f2203 to f5fb609 Compare July 22, 2025 08:17
@eyeinsky

Copy link
Copy Markdown
Contributor

The CI failed for yesterday's changes, a few of these I think were unrelated to what I did (fake-aws-sqs timed out, upload-hoogle-nix failed), and with one I broke the linter -- the linter thing I fixed and now waiting on CI.

Comment thread services/galley/src/Galley/API/MLS/Proposal.hs Outdated
@pcapriotti
pcapriotti merged commit ca47e49 into develop Jul 24, 2025
9 checks passed
@pcapriotti
pcapriotti deleted the group-info-check branch July 24, 2025 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants