Skip to content

Add Android Automotive sample#2724

Merged
0xnm merged 2 commits into
developfrom
xgouchet/automotive
Jun 16, 2025
Merged

Add Android Automotive sample#2724
0xnm merged 2 commits into
developfrom
xgouchet/automotive

Conversation

@0xnm

@0xnm 0xnm commented Jun 12, 2025

Copy link
Copy Markdown
Member

What does this PR do?

Same as #2722, but branch was moved into this repo.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm requested review from a team as code owners June 12, 2025 14:28
xgouchet
xgouchet previously approved these changes Jun 12, 2025
@0xnm
0xnm force-pushed the xgouchet/automotive branch 2 times, most recently from a5e7405 to 10669b9 Compare June 12, 2025 14:38
@0xnm
0xnm requested a review from xgouchet June 12, 2025 14:38
private fun createDatadogConfiguration(): Configuration {
return Configuration.Builder(
clientToken = BuildConfig.DD_CLIENT_TOKEN,
env = "test",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we don't use BuildConfig.BUILD_TYPE here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@xgouchet Can you please check this one and comment below?

return Configuration.Builder(
clientToken = BuildConfig.DD_CLIENT_TOKEN,
env = "test",
variant = ""

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice to have BuildConfig.FLAVOR here

@0xnm
0xnm force-pushed the xgouchet/automotive branch from 10669b9 to 5b82786 Compare June 12, 2025 15:10
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.78%. Comparing base (dc4ab79) to head (5b82786).
Report is 19 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2724      +/-   ##
===========================================
- Coverage    69.85%   69.78%   -0.07%     
===========================================
  Files          823      823              
  Lines        30626    30626              
  Branches      5157     5157              
===========================================
- Hits         21391    21371      -20     
- Misses        7785     7804      +19     
- Partials      1450     1451       +1     

see 33 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@0xnm
0xnm merged commit f583493 into develop Jun 16, 2025
25 checks passed
@0xnm
0xnm deleted the xgouchet/automotive branch June 16, 2025 07:14
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.

4 participants