-
Notifications
You must be signed in to change notification settings - Fork 133
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.
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
chore: various fixes to build with SPI #1812
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 3083 Passed, 0 Skipped, 30m 1.65s Wall Time New Flaky Tests (2)
🔻 Code Coverage Decreases vs Default Branch (10)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few minor questions but otherwise looks good.
DatadogSessionReplay/Tests/Recorder/Utilties/ImageDataProviderTests.swift
Outdated
Show resolved
Hide resolved
...r/ViewTreeSnapshotProducer/ViewTreeSnapshot/NodeRecorders/UnsupportedViewRecorderTests.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just some minor comments
What and why?
https://swiftpackageindex.com/DataDog/dd-sdk-ios/builds
How?
The main issue here was with tvOS compile out flag, where types were not included but their tests were part of the target.
SPI compiles the project without xcworkspace ie it deletes the workspace which forces Xcode to automatically generate the Datadog-Package scheme.
now following command builds
Review checklist
Custom CI job configuration (optional)
tools/