Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat(bridge): alert error through pagerduty #98

Merged
merged 1 commit into from
Sep 30, 2021

Conversation

moreal
Copy link
Collaborator

@moreal moreal commented Sep 24, 2021

This resolves #96.

@moreal moreal self-assigned this Sep 24, 2021
@moreal moreal added the bridge An issue or a pull request related to bridge label Sep 24, 2021
@codecov
Copy link

codecov bot commented Sep 24, 2021

Codecov Report

Merging #98 (21f0a0e) into main (f0ba916) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
+ Coverage   94.93%   95.00%   +0.06%     
==========================================
  Files          18       18              
  Lines         316      320       +4     
  Branches       43       43              
==========================================
+ Hits          300      304       +4     
  Misses         10       10              
  Partials        6        6              
Impacted Files Coverage Δ
bridge/src/observers/burn-event-observer.ts 100.00% <100.00%> (ø)
bridge/src/observers/nine-chronicles.ts 94.87% <100.00%> (+0.13%) ⬆️

@moreal moreal requested a review from longfin September 24, 2021 07:11
@moreal moreal merged commit cbe992f into planetarium:main Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bridge An issue or a pull request related to bridge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate PagerDuty for ops
3 participants