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

Jump to bottom

RUM-3461 feat: Fatal App Hangs monitoring #1763

Merged
merged 11 commits into from
Apr 10, 2024
Merged

RUM-3461 feat: Fatal App Hangs monitoring #1763

merged 11 commits into from
Apr 10, 2024

Conversation

ncreated
Copy link
Member

@ncreated ncreated commented Apr 5, 2024 •
edited
Loading

What and why?

📦 Merging the fatal App Hangs monitoring feature.

How?

CHANGELOG.md update + merge of following work (no code review required):

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing changes

Custom CI job configuration (optional)

  • Run unit tests for Core, RUM, Trace, Logs, CR and WVT
  • Run unit tests for Session Replay
  • Run integration tests
  • Run smoke tests
  • Run tests for tools/

@ncreated ncreated self-assigned this Apr 5, 2024
ncreated added 5 commits April 5, 2024 10:50
@ncreated
Merge branch 'develop' into ncreated/RUM-3461/fatal-app-hangs
a63cde9 
* develop:
  Remove core reference
  RUM-2813 Add support for internal_telemetry from browser
  doc: Add pull request links in CHANGELOG
  Fix inconsistency with null returns from mapper
  Don't allow dropping error events from crash mapper
  feat: Allow crash reporting to use event mappers
  Remove module map
@ncreated
RUM-3461 Send fatal App Hang after app restart
4edfde2
@ncreated
RUM-3461 Test data and conditions behind App Hang uploads
66a117d
@ncreated
RUM-3461 Update CHANGELOG.md
a3b0263
@ncreated
RUM-3461 Lint
9515f36
@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Apr 5, 2024 •
edited
Loading

Datadog Report

Branch report: ncreated/RUM-3461/fatal-app-hangs
Commit report: a3b0263
Test service: dd-sdk-ios

✅ 0 Failed, 2932 Passed, 0 Skipped, 12m 26.38s Wall Time
🔻 Test Sessions change in coverage: 9 decreased, 4 increased, 1 no change

🔻 Code Coverage Decreases vs Default Branch (9)

This report shows up to 5 code coverage decreases.

  • test DatadogTraceTests tvOS 49.09% (-0.15%) - Details
  • test DatadogInternalTests tvOS 78.96% (-0.15%) - Details
  • test DatadogWebViewTrackingTests iOS 17.67% (-0.14%) - Details
  • test DatadogTraceTests iOS 49.06% (-0.13%) - Details
  • test DatadogLogsTests iOS 45.37% (-0.11%) - Details

@ncreated ncreated marked this pull request as ready for review April 10, 2024 13:02
@ncreated ncreated requested review from a team as code owners April 10, 2024 13:02
@ncreated ncreated merged commit 09b7893 into develop Apr 10, 2024
8 checks passed
@ncreated ncreated deleted the ncreated/RUM-3461/fatal-app-hangs branch April 10, 2024 13:04
This was referenced Apr 16, 2024
Merged
Merged
@mariedm mariedm mentioned this pull request Dec 20, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxep maxep maxep approved these changes

Assignees

@ncreated ncreated

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ncreated @maxep