search

kurotych / sqlant

Generate PlantUML/Mermaid ER diagram textual description from SQL connection string
MIT License
22 stars 1 forks source link

Replace panic and unwrap with Result #18

Closed jayvdb closed 5 months ago

jayvdb commented 6 months ago

Continuing from https://github.com/kurotych/sqlant/pull/17 , there are a lot of panic & unwrap in the lib code.

IMO getting rid of these should be a priority, as adding Result everywhere breaks the API, so now is a good time to do those breaking changes.