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

copying scope data to the event

💡 Motivation and Context

scope data was not being coped to the event

💚 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-io commented Oct 25, 2019

Codecov Report

Merging #68 into master will increase coverage by 2.71%.
The diff coverage is 88.88%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master     #68      +/-   ##
===========================================
+ Coverage     48.79%   51.5%   +2.71%     
- Complexity      281     314      +33     
===========================================
  Files            56      56              
  Lines          1494    1526      +32     
  Branches         94     106      +12     
===========================================
+ Hits            729     786      +57     
+ Misses          730     701      -29     
- Partials         35      39       +4
Impacted Files Coverage Δ Complexity Δ
...core/src/main/java/io/sentry/core/SentryEvent.java 55.91% <75%> (+25.32%) 30 <3> (+17) ⬆️
...ore/src/main/java/io/sentry/core/SentryClient.java 86.88% <95.83%> (+5.8%) 22 <0> (+11) ⬆️
...re/src/main/java/io/sentry/core/protocol/User.java 38.88% <0%> (+16.66%) 5% <0%> (+2%) ⬆️
...entry-core/src/main/java/io/sentry/core/Scope.java 86.2% <0%> (+17.24%) 15% <0%> (+3%) ⬆️

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 8d03bb7...1d7e714. Read the comment docs.

@bruno-garcia bruno-garcia merged commit 3efad28 into master Oct 27, 2019
@bruno-garcia bruno-garcia deleted the feat/scope_to_event branch October 27, 2019 20:55
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.

4 participants