Improved error handler in package util and player. Changed log.Fatal to log.Panic on player.go.

alvarorichard / GoAnime

A CLI tool to browse, play, and download anime in pt-br (Portuguese)

MIT License

122 stars 8 forks source link

Improved error handler in package util and player. Changed log.Fatal to log.Panic on player.go. #40

Closed the-eduardo closed 8 months ago

the-eduardo commented 8 months ago

Should a Go package ever use log.Fatal?

log.Fatal: Logs the message and terminates the program immediately with os.Exit(1). Deferred functions will not run.

log.Panic: Logs the message and then panics. This allows deferred functions to run.

deepsource-io[bot] commented 8 months ago

Here's the code health analysis summary for commits 350a719..82adcf5. View details on DeepSource ↗.

Analysis Summary

Analyzer	Status	Summary	Link
Shell	✅ Success		View Check ↗
Go	✅ Success		View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.