Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Revamp xcframework generation to avoid conflicts with other Rust libraries #742

Merged

Conversation

aringenbach
Copy link
Contributor

Hopefully the move to a documented shell script makes all the whacky things we have to do for the iOS generation a bit clearer.

@sonarcloud
Copy link

sonarcloud bot commented Jul 19, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov-commenter
Copy link

Codecov Report

Patch coverage has no change and project coverage change: -0.23 ⚠️

Comparison is base (69e0e48) 90.17% compared to head (1486d5f) 89.94%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #742      +/-   ##
==========================================
- Coverage   90.17%   89.94%   -0.23%     
==========================================
  Files         113       83      -30     
  Lines       16120    14678    -1442     
  Branches      565        0     -565     
==========================================
- Hits        14536    13202    -1334     
+ Misses       1563     1476      -87     
+ Partials       21        0      -21     
Flag Coverage Δ
uitests ?
uitests-ios ?
unittests 89.94% <ø> (+0.87%) ⬆️
unittests-ios ?
unittests-rust 89.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 30 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor

@langleyd langleyd left a comment

Choose a reason for hiding this comment

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

Nice, LGTM

@aringenbach aringenbach merged commit 3a4686f into main Jul 20, 2023
7 checks passed
@aringenbach aringenbach deleted the aringenbach/fix-conflicts-with-multiple-rust-libraries branch July 20, 2023 10:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants