This cannot be the real secret api key?
https://github.com/eoin-barr/weatherme/blob/master/cmd/root....
return err if err != nil
everything better than three fucking lines every time something returns err.Especially that some clown decided that apparently
if err != nil { return err }
is too terse and go fmt just goes back to expanding it to 3 lines...Don't get me wrong, error handling is verbose and annoying, but not even displaying actual error is just horrible coding practice.
should be at the very least
err = json.Unmarshal(weatherBody, &WeatherRes)
if err != nil {
fmt.Printf("error decoding weather json: %s\n",err)
return
}
or just be function that returns that error or the weather err = json.Unmarshal(weatherBody, &WeatherRes)
if err != nil {
return fmt.Errorf("error decoding weather json: %s\n",err)
}