Skip to content

Reduce useless io calls #4333

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Reduce useless io calls #4333

wants to merge 2 commits into from

Conversation

stefanosiano
Copy link
Member

📜 Description

Reduce useless io calls, to mitigate StrictMode warnings.

💡 Motivation and Context

Relates to #4261

💚 How did you test it?

📝 Checklist

  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

Copy link
Contributor

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- Reduce useless io calls ([#4333](https://github.com/getsentry/sentry-java/pull/4333))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by 🚫 dangerJS against a52a423

Copy link
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 429.90 ms 450.76 ms 20.86 ms
Size 1.58 MiB 2.08 MiB 505.43 KiB

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant