mountainMath / cancensus

R wrapper for calling CensusMapper APIs
https://mountainmath.github.io/cancensus/index.html
Other
82 stars 15 forks source link

Migrate options() to environment variables as default #160

Closed mountainMath closed 3 years ago

mountainMath commented 3 years ago

Right now we set the API key and cache path via options. It might be cleaner to set these as environment variables by default instead. Nothing would change in terms of user experience, but we would change the defaults in the function calls and also change the docs, but fall back to options in case environment variables aren't set. And also change the docs and vignette accordingly.

mountainMath commented 3 years ago

Addressed in https://github.com/mountainMath/cancensus/tree/intersect_geometry2

mountainMath commented 3 years ago

Done and merged into main.