Skip to content

feat: API changes - #1497

Merged
HazAT merged 12 commits into
masterfrom
ref/api-changes
Sep 3, 2018
Merged

feat: API changes#1497
HazAT merged 12 commits into
masterfrom
ref/api-changes

Conversation

@HazAT

@HazATHazAT commented Aug 30, 2018

Copy link
Copy Markdown
Member
  • Add hint parameter for error processing
  • Remove eventprocessors from hub (now only on scope)
  • Remove shouldSendafterSend, shouldAddBreadcrumb, afterBreadcrumb

@HazATHazAT self-assigned this Aug 30, 2018
@codecov-io

codecov-io commented Aug 31, 2018

Copy link
Copy Markdown

Codecov Report

Merging #1497 into master will decrease coverage by 0.19%.
The diff coverage is 86.02%.

Impacted file tree graph

@@ Coverage Diff @@## master #1497 +/- ##
=========================================
- Coverage 72.61% 72.42% -0.2% 
=========================================
Files 41 41 Lines 1249 1262 +13 Branches 239 241 +2 =========================================
+ Hits 907 914 +7 - Misses 333 339 +6 
Partials 9 9
Impacted FilesCoverage Δ
packages/hub/src/hub.ts98.48% <100%> (+0.12%)⬆️
packages/hub/src/scope.ts95.18% <100%> (ø)⬆️
packages/core/test/mocks/backend.ts100% <100%> (ø)⬆️
...ages/node/src/integrations/onunhandledrejection.ts100% <100%> (ø)⬆️
packages/node/src/handlers.ts21.97% <14.28%> (-0.5%)⬇️
packages/node/src/integrations/sdkinformation.ts88.88% <75%> (+1.38%)⬆️
packages/node/src/backend.ts73.91% <80%> (+1.82%)⬆️
packages/node/src/integrations/clientoptions.ts90.9% <85.71%> (+0.9%)⬆️
packages/minimal/src/index.ts95.83% <91.66%> (-4.17%)⬇️
packages/core/src/base.ts91.95% <95.65%> (-3.51%)⬇️

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 d54c762...2423b24. Read the comment docs.

@HazATHazAT changed the title [WIP] feat: API changesfeat: API changesAug 31, 2018
@HazAT
HazAT merged commit f120f1a into masterSep 3, 2018
@HazAT
HazAT deleted the ref/api-changes branch September 3, 2018 07:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@HazAT@codecov-io@kamilogorek