Skip to content

Conversation

@Mansi-mParticle
Copy link
Collaborator

Background

  • The SDK currently uses a mix of logging approaches, including printStackTrace(), which is not aligned with our recommended logging pattern. As noted in the internal guidance, all developer-facing logs should go through the Logger class to ensure consistency and proper log handling.

What Has Changed

  • Replaced e.printStackTrace() with Logger.error(e) to follow the standard logging approach.

Screenshots/Video

  • {Include any screenshots or video demonstrating the new feature or fix, if applicable}

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have tested this locally.

Additional Notes

  • {Any additional information or context relevant to this PR}

Reference Issue (For employees only. Ignore if you are an outside contributor)

@Mansi-mParticle Mansi-mParticle requested a review from a team as a code owner December 4, 2025 21:43
@Mansi-mParticle Mansi-mParticle force-pushed the fix/SDKE-674-migrate-generic-logs-to-logger branch from ec87e70 to 64867e2 Compare December 4, 2025 21:59
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 5, 2025

thomson-t
thomson-t previously approved these changes Dec 5, 2025
jamesnrokt
jamesnrokt previously approved these changes Dec 9, 2025
Signed-off-by: Mansi-mParticle <159845845+Mansi-mParticle@users.noreply.github.com>
@Mansi-mParticle Mansi-mParticle dismissed stale reviews from jamesnrokt and thomson-t via d65b8e8 January 13, 2026 16:30
@sonarqubecloud
Copy link

@Mansi-mParticle Mansi-mParticle merged commit 9c9cb33 into main Jan 13, 2026
17 checks passed
@Mansi-mParticle Mansi-mParticle deleted the fix/SDKE-674-migrate-generic-logs-to-logger branch January 13, 2026 16:47
mparticle-automation added a commit that referenced this pull request Jan 27, 2026
## [5.76.0](v5.75.1...v5.76.0) (2026-01-27)

### Features

* Add trunk rule to detect mParticle API keys ([#633](#633)) ([3f3c79f](3f3c79f))
* Update Android Gradle plugin to 8.1.0 ([#575](#575)) ([fc6a25d](fc6a25d))

### Bug Fixes

* Align ktlint rules for kits with Android Studio defaults ([#640](#640)) ([e5484a0](e5484a0))
* migrate all generic logs to Logger class ([#630](#630)) ([9c9cb33](9c9cb33))
* Override ktlint rules for kits to avoid editorconfig conflicts ([#642](#642)) ([e31027a](e31027a))

### Updates & Maintenance

* bump actions/cache from 4 to 5 ([#635](#635)) ([fb471fa](fb471fa))
* bump actions/checkout from 5 to 6 ([#631](#631)) ([a0b2442](a0b2442))
* bump actions/checkout from 5.0.0 to 6.0.1 ([#637](#637)) ([449a0b5](449a0b5))
* bump actions/setup-java from 4 to 5 ([#625](#625)) ([46823b8](46823b8))
* bump actions/upload-artifact from 5 to 6 ([#636](#636)) ([05f6133](05f6133))
* Remove empty workflow file ([#634](#634)) ([0780abd](0780abd))
* Update submodules ([d6bf37f](d6bf37f))
@mparticle-automation
Copy link
Collaborator

🎉 This PR is included in version 5.76.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants