While not too bad yet, there are only two working providers (and a test which piggybacks on one of the two) taking up >500 lines of code. Simply, the current files should be more modular to allow for easier updates and additions.
Particularly:
each provider should have their own file exposing the information we need in a standard format (e.g. the URI to call, if an API key is needed, etc.)
weather-aware.js should be the single interface from other code wanting to get the weather. Within the file, it should handle gathering all the information from the provider code and try to ensure the weather data is returned
While not too bad yet, there are only two working providers (and a test which piggybacks on one of the two) taking up >500 lines of code. Simply, the current files should be more modular to allow for easier updates and additions.
Particularly:
weather-aware.jsshould be the single interface from other code wanting to get the weather. Within the file, it should handle gathering all the information from the provider code and try to ensure the weather data is returned