Skip to content

Use BloomFilter.xcframework binary dependency#1269

Merged
mallexxx merged 5 commits into
mainfrom
alex/bloom-cpp-xcframework
Jul 17, 2025
Merged

Use BloomFilter.xcframework binary dependency#1269
mallexxx merged 5 commits into
mainfrom
alex/bloom-cpp-xcframework

Conversation

@mallexxx
Copy link
Copy Markdown
Contributor

@mallexxx mallexxx commented Jul 7, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/1203301625297703/task/1210725543567479?focus=true
Bloom_cpp PR: duckduckgo/bloom_cpp#27

Description

  • Replace BloomFilter dependency with xcframework

Testing Steps

  1. Validate BSK/CI is green for iOS/macOS
  2. Validate HTTPS upgrade using bloom filter works

Impact and Risks

  • High

What could go wrong?

  • Runtime crashes on navigation

Quality Considerations

Notes to Reviewer


Internal references:

Definition of Done | Engineering Expectations | Tech Design Template

@mallexxx mallexxx requested a review from ayoy July 9, 2025 04:45
Copy link
Copy Markdown
Collaborator

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

Looks great, thanks so much for doing this!

Copy link
Copy Markdown
Collaborator

@ayoy ayoy left a comment

Choose a reason for hiding this comment

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

Looks great indeed, thanks @mallexxx!

mallexxx added a commit to duckduckgo/bloom_cpp that referenced this pull request Jul 17, 2025
@mallexxx mallexxx merged commit 22b6f1a into main Jul 17, 2025
36 checks passed
@mallexxx mallexxx deleted the alex/bloom-cpp-xcframework branch July 17, 2025 10:45
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.

3 participants