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.

Sign up for GitHub

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

Jump to bottom

Fix panic in `PartialMmr::untrack` #382

Merged

bobbinth merged 2 commits into main from pgackst-partial-mmr-untrack

Feb 18, 2025

Contributor

PhilippGackstatter commented Feb 18, 2025

Describe your changes

Fix panic in PartialMmr::untrack and add a test for it.

closes #379

Checklist before requesting a review

Repo forked and branch created from next according to naming convention.
Commit messages and codestyle follow conventions.
Relevant issues are linked in the PR description.
Tests added for new functionality.
Documentation/comments updated according to changes.

PhilippGackstatter added 2 commits

February 18, 2025 09:51


          fix: PartialMmr::untrack and test it

73fe075


          chore: Add changelog entry

a19b132

sonarqubecloud bot commented Feb 18, 2025

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

PhilippGackstatter requested a review from bobbinth

February 18, 2025 09:01

bobbinth approved these changes

View reviewed changes

Contributor

bobbinth left a comment

Looks good! Thank you!

bobbinth merged commit 535637d into main

13 checks passed

bobbinth deleted the pgackst-partial-mmr-untrack branch

February 18, 2025 09:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet