Skip to content

refactor(analytics): Only plant DebugTree in debug builds#4025

Merged
jamesarich merged 1 commit into
meshtastic:mainfrom
jamesarich:chore/debug-tree
Dec 17, 2025
Merged

refactor(analytics): Only plant DebugTree in debug builds#4025
jamesarich merged 1 commit into
meshtastic:mainfrom
jamesarich:chore/debug-tree

Conversation

@jamesarich

Copy link
Copy Markdown
Collaborator

The DebugTree for Timber logging will now only be planted when the application is built in DEBUG mode. This is achieved by conditionally adding it to the list of trees to be planted based on the BuildConfig.DEBUG flag.

The `DebugTree` for Timber logging will now only be planted when the application is built in `DEBUG` mode. This is achieved by conditionally adding it to the list of trees to be planted based on the `BuildConfig.DEBUG` flag.

Signed-off-by: James Rich <2199651+jamesarich@users.noreply.github.com>
@jamesarich jamesarich enabled auto-merge December 17, 2025 16:01
@jamesarich jamesarich added this pull request to the merge queue Dec 17, 2025
@codecov

codecov Bot commented Dec 17, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.56%. Comparing base (380de5e) to head (f6e1aca).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...core/analytics/platform/GooglePlatformAnalytics.kt 0.00% 6 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #4025      +/-   ##
========================================
- Coverage   0.56%   0.56%   -0.01%     
========================================
  Files        405     405              
  Lines      24136   24141       +5     
  Branches    3098    3099       +1     
========================================
  Hits         136     136              
- Misses     23979   23984       +5     
  Partials      21      21              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Merged via the queue into meshtastic:main with commit 506cba2 Dec 17, 2025
6 checks passed
@jamesarich jamesarich deleted the chore/debug-tree branch December 17, 2025 16:39
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.

1 participant