Skip to content
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

ref(profiling): extract methods related to profile serialization #3876

Merged
merged 16 commits into from
Apr 25, 2024

Revert "move to test util target"

0b3796f
Select commit
Loading
Failed to load commit list.
Merged

ref(profiling): extract methods related to profile serialization #3876

Revert "move to test util target"
0b3796f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Apr 23, 2024 in 0s

89.142% of diff hit (target 90.632%)

View this Pull Request on Codecov

89.142% of diff hit (target 90.632%)

Annotations

Check warning on line 60 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L60 

Added line #L60 was not covered by tests

Check warning on line 69 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L69 

Added line #L69 was not covered by tests

Check warning on line 113 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L111-L113 

Added lines #L111 - L113 were not covered by tests

Check warning on line 122 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L120-L122 

Added lines #L120 - L122 were not covered by tests

Check warning on line 207 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L207 

Added line #L207 was not covered by tests

Check warning on line 229 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L229 

Added line #L229 was not covered by tests

Check warning on line 244 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L244 

Added line #L244 was not covered by tests

Check warning on line 257 in Sources/Sentry/Profiling/SentryProfilerSerialization.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerSerialization.mm#L257 

Added line #L257 was not covered by tests
View more details on Codecov