dolthub / dolthub/dolt

Replace panics with error messages

Open
#11,299 1 comment 0 reactions 0 assignees View on GitHub
bad error message enhancement panic
Dominant language
Go
Stars
24.4k
Forks
873
Avg merge
1d 5h
Merged PRs (30d)
108

Description

There are a lot of places in Dolt and GMS where we intentionally panic when encountering unexpected values. This makes for a bad user experience. Instead, we should error out with a message.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by surveying the panic sites across Dolt and GMS, since the issue does not identify particular files, tests, or entry points. Replace intentional panics for unexpected values with user-facing error messages, and consider the work done when those paths error out instead of panicking.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
databases
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.