Skip to content
This repository was archived by the owner on Aug 30, 2023. It is now read-only.

Conversation

@marandaneto
Copy link
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Added the option via Manifest conf. to disable the auto-init of the Sentry SDK, so before fully initing the SDK we check this flag and don't do anything, the user may init. using a custom Application class.

💡 Motivation and Context

the user may want to init. the SDK by code and pass custom parameters based on buildType/flavors and so on.

💚 How did you test it?

Mocking the manifest value and unit testing.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

This should be written in the setup documentation

@codecov-io
Copy link

codecov-io commented Oct 14, 2019

Codecov Report

Merging #30 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #30   +/-   ##
=========================================
  Coverage     13.61%   13.61%           
  Complexity       66       66           
=========================================
  Files            36       36           
  Lines           764      764           
  Branches         18       18           
=========================================
  Hits            104      104           
  Misses          658      658           
  Partials          2        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 587ed63...7db1dca. Read the comment docs.

@marandaneto marandaneto merged commit 8c1991e into master Oct 14, 2019
@marandaneto marandaneto deleted the feat/opt_auto_init branch October 14, 2019 14:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants