Skip to content

chore: Remove the matrix-sdk-crypto re-export in the matrix-sdk crate#5769

Merged
poljar merged 1 commit intomainfrom
poljar/remove-crypto-re-export
Oct 24, 2025
Merged

chore: Remove the matrix-sdk-crypto re-export in the matrix-sdk crate#5769
poljar merged 1 commit intomainfrom
poljar/remove-crypto-re-export

Conversation

@poljar
Copy link
Copy Markdown
Contributor

@poljar poljar commented Oct 10, 2025

  • Public API changes documented in changelogs (optional)

@poljar poljar requested a review from a team as a code owner October 10, 2025 09:06
@poljar poljar requested review from stefanceriu and removed request for a team October 10, 2025 09:06
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Oct 10, 2025

CodSpeed Performance Report

Merging #5769 will not alter performance

Comparing poljar/remove-crypto-re-export (126e02a) with main (d117532)

Summary

✅ 50 untouched

@poljar poljar requested review from richvdh and removed request for stefanceriu October 10, 2025 09:28
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.46%. Comparing base (d117532) to head (126e02a).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5769      +/-   ##
==========================================
- Coverage   88.46%   88.46%   -0.01%     
==========================================
  Files         360      360              
  Lines      100412   100412              
  Branches   100412   100412              
==========================================
- Hits        88833    88830       -3     
- Misses       7413     7417       +4     
+ Partials     4166     4165       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

sure

@poljar poljar force-pushed the poljar/remove-crypto-re-export branch from 48b2f94 to 211e446 Compare October 24, 2025 14:09
@poljar poljar force-pushed the poljar/remove-crypto-re-export branch from 211e446 to 126e02a Compare October 24, 2025 14:18
@poljar poljar enabled auto-merge (rebase) October 24, 2025 14:19
@poljar poljar merged commit 3733ee8 into main Oct 24, 2025
53 checks passed
@poljar poljar deleted the poljar/remove-crypto-re-export branch October 24, 2025 14:37
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