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

Javadocs for SentryOptions #246

Merged
merged 2 commits into from
Jan 27, 2020
Merged

Javadocs for SentryOptions #246

merged 2 commits into from
Jan 27, 2020

Conversation

marandaneto
Copy link
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Javadocs for SentryOptions.

💡 Motivation and Context

Users may learn directly from code.

💚 How did you test it?

📝 Checklist

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

🔮 Next steps

@codecov-io
Copy link

Codecov Report

Merging #246 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #246   +/-   ##
=========================================
  Coverage     57.27%   57.27%           
  Complexity      567      567           
=========================================
  Files            72       72           
  Lines          2708     2708           
  Branches        234      234           
=========================================
  Hits           1551     1551           
  Misses         1042     1042           
  Partials        115      115
Impacted Files Coverage Δ Complexity Δ
...src/main/java/io/sentry/core/DiagnosticLogger.java 72.22% <ø> (ø) 11 <0> (ø) ⬇️
...re/src/main/java/io/sentry/core/SentryOptions.java 79.64% <100%> (ø) 55 <0> (ø) ⬇️

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 4cb55c2...051a042. Read the comment docs.

@bruno-garcia bruno-garcia merged commit fd6413b into master Jan 27, 2020
@bruno-garcia bruno-garcia deleted the enha/javadocs branch January 27, 2020 18:51
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.

3 participants