Skip to content

Feature/datadog crash logging - #213

Draft
ninovanhooff wants to merge 11 commits into
mainfrom
feature/datadog-crash-logging
Draft

ninovanhooff wants to merge 11 commits into
mainfrom
feature/datadog-crash-logging

Conversation

@ninovanhooff

@ninovanhooff ninovanhooff commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Why is this important?

For the techops for apps workgroup we are exploring Datadog. This PR is not meant do be merged, just as a poc.
We probably will be using Kermit + Crashlytics as default.

  • create an interface for logging so that unit tests don't have to depend on Datadog (which is a problem for iOS)
  • replace Kermit logging by Datadog
  • replace Crashlytics by Datadog
  • remove firebase since it is not used anymore

Notes

Considerations in case we want to merge this to main sometime

  • maybe only merge the part where logging is hidden behind an interface, so datadog does not become the default but it is more easily added
  • where to put the secrets? currently in the repo (shared/build.gradle.kts)
  • For what branches to run the workflows and dsym upload steps

feature/datadog-crash-logging
Completely removes firebase since it is not used anymore.
Crashlytics was removed because of reports stating that crash reporting libraries can fight among eachother
@ninovanhooff
ninovanhooff changed the base branch from main to chore/139-remove-jvm-target September 3, 2026 11:11
Base automatically changed from chore/139-remove-jvm-target to main September 4, 2026 09:46
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