rOpenGov / rtimes

R wrapper for NYTimes API for government data - ABANDONED

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Storing token in environment rather than global options

soodoku opened this issue · comments

Dear Scott,

A minor suggestion. @hadley recommends storing tokens in environment rather than options. It makes sense to me. Options are global afaik. And we probably want something like env.

https://cran.r-project.org/web/packages/httr/vignettes/api-packages.html

Hope you are well.

Gaurav

yeah, can change, I do use env vars for keys for travis testing e.g., https://github.com/rOpenGov/rtimes/blob/master/R/zzz.R#L41 - just not for users, but will change