briandowns/openweathermap

Return error instead of using log.Fatalln

Closed this issue · 2 comments

Error should be returned in the case of calls like DailyByName, etc. Exiting because something cannot be parsed is not the right behavior.

I see what you mean. The forecast code is kind of a mess. I can probably get to this sometime today unless you had a PR planned and I'll gladly wait.

This has been fixed in #3 .