Add ability to construct a new WeatherAPI instance with credentials about weatherapi HOT 4 OPEN

Does the changes introduced in 3454682 solve this for you?

from weatherapi.

It's definitely more flexible but I'd prefer an explicit constructor:

WeatherAPI instance = new WeatherAPI(credentials, lruCacheSize, etc)

from weatherapi.

My only reservation with an explicit constructor is all the overloading that would need to be done for different sets of parameters. Some kind of config object that can be passed in would solve this, however.

from weatherapi.

As a quick heads-up - Snowplow wintern @AALEKH is working on a PR to add this functionality. We need this to deliver the weather enrichment in Snowplow...

from weatherapi.