Fix weather fetcher#44

Open

scorphus wants to merge 2 commits into

oh-my-fish:masterfrom

scorphus:fix-weather-fetcher

Member

scorphus commented Dec 27, 2022

This PR introduces two changes:

Fail weather.fetch with no data on HTTP errors
Fix When $argv is empty null errors apear #35
Close test if $argv is set to prevent weather to run an empty query #36
Fix Math Errors on Install #42
Close Use documented API parameter #43
Fix weather.fetcher identifying stale cache files
Fix Update weather 'cache' for city command? #34

scorphus added 2 commits

December 27, 2022 21:43


          Fix weather.fetcher identifying stale cache files

f9b2556

Fix oh-my-fish#34


          Fail weather.fetch with no data on HTTP errors

a9ae48d

Fix oh-my-fish#35
Close oh-my-fish#36
Fix oh-my-fish#42
Close oh-my-fish#43

scorphus requested review from a team, bobthecow and derekstavis

December 27, 2022 22:52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet