duneanalytics / dune-client

A framework for interacting with Dune Analytics' officially supported API service
Apache License 2.0
85 stars 22 forks source link

Add Client.from_env constructor #62

Closed bh2smith closed 1 year ago

bh2smith commented 1 year ago

This will simplify the user experience, not needing to import env vars manually (putting it all under the hood). Should pave the way for making documentation easier.

Closes #68

cc @TheEdgeOfRage & @diegoximenes for additional review.

bh2smith commented 1 year ago

I have addressed all comments and suggestions. No more load_dotenv package dependency.

bh2smith commented 1 year ago

I am going to assume this is alright since I have addressed the comments & suggestions made by @msf. I'll wait a bit and merge tomorrow morning.