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.

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

Dogfood recent changes #1865

Merged
merged 99 commits into from
May 29, 2024
Merged

Dogfood recent changes #1865

merged 99 commits into from
May 29, 2024

Conversation

ncreated
Copy link
Member

What and why?

How?

developdogfooding → 🤖

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 and others added 30 commits May 14, 2024 09:35
Co-authored-by: Ursula Chen <[email protected]>
…ate-on-attributes-change-part1

RUM-3588 chore: Add tests for global RUM attributes
Co-authored-by: Maciek Grzybowski <[email protected]>
…encoder and decoder

+ simplify ISO8601 date coding
As observed in #1638, accessing `request.allHTTPHeaderFields` is not
safe and leads to crashes with undefined root cause. To avoid this, instead we use `request.value(forHTTPHeaderField:)`
to only read headers known by the SDK.
ncreated and others added 18 commits May 21, 2024 15:56
…racking-for-obj-c

RUM-4313 feat: Add `WebViewTracking` for Obj-c
…um-models

chore: Fix generating rum models
…context-with-rum-attributes

RUM-3588 fix: Always update crash context with RUM attributes
…with-swift-testing

[CIVIS-9813] Fix for test sdk integration
…ntext-control

RUM-3535 feat: add support for trace context injection configuration to allow selective injection
…accessing-allHTTPHeaderFields

RUM-3183 fix: Fix crash on accessing `request.allHTTPHeaderFields`
…or telemetry

this is to later aggregate SDK errors by their kind in "RUM Session Ended" telemetry
…-oom-crash

RUM-3464 feat(oom): simulate OOM crash in Example app
…metrics

refactor: Standardize locating SDK metrics code
@ncreated ncreated requested review from a team as code owners May 29, 2024 07:07
@ncreated ncreated self-assigned this May 29, 2024
@ncreated ncreated removed the request for review from a team May 29, 2024 07:08
@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented May 29, 2024

Datadog Report

Branch report: develop
Commit report: 19d14bb
Test service: dd-sdk-ios

✅ 0 Failed, 3222 Passed, 0 Skipped, 2m 32.5s Total Time
➡️ Test Sessions change in coverage: 13 no change

Copy link
Member

@maxep maxep left a comment

Choose a reason for hiding this comment

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

🐕

@ncreated
Copy link
Member Author

CI is green, Bitrise checks are false. Moving forward with bypassing checks for merge.

Screenshot 2024-05-29 at 11 36 46

@ncreated ncreated merged commit b05d123 into dogfooding May 29, 2024
7 of 9 checks passed
@ncreated ncreated mentioned this pull request May 29, 2024
8 tasks
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.

5 participants