Skip to content

Commit

Permalink
Removed sentry since they should already be captured
Browse files Browse the repository at this point in the history
  • Loading branch information
dylanratcliffe committed Jul 19, 2024
1 parent cf3f253 commit f3428d7
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions cmd/tea_submitplan.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ import (
"connectrpc.com/connect"
tea "github.com/charmbracelet/bubbletea"
"github.com/charmbracelet/lipgloss"
"github.com/getsentry/sentry-go"
"github.com/google/uuid"
"github.com/muesli/reflow/wordwrap"
"github.com/overmindtech/sdp-go"
Expand Down Expand Up @@ -681,9 +680,6 @@ func (m submitPlanModel) submitPlanCmd() tea.Msg {
m.processing <- submitPlanUpdateMsg{m.blastRadiusTask.UpdateStatusMsg(taskStatusError)}
m.processing <- submitPlanUpdateMsg{m.risksError("failed to update planned changes", err)}
close(m.processing)

// TODO: Ask David about the position of these too
sentry.CaptureException(err)
return nil
}

Expand Down Expand Up @@ -723,9 +719,6 @@ func (m submitPlanModel) submitPlanCmd() tea.Msg {
m.processing <- submitPlanUpdateMsg{m.blastRadiusTask.UpdateStatusMsg(taskStatusError)}
m.processing <- submitPlanUpdateMsg{m.risksError("error streaming results", err)}
close(m.processing)

// TODO: Ask David about the position of these too
sentry.CaptureException(err)
return nil
}
m.processing <- submitPlanUpdateMsg{m.blastRadiusTask.FinishMsg()}
Expand Down

0 comments on commit f3428d7

Please sign in to comment.