Environment variables names too generic
Thanks for the package, really convenient and useful!
The only issue I have is the naming of the environment variables, EMAIL_ADDRESS and ACCESS_KEY. I think best practice in packages is to have a clear prefix, in this case maybe ACLED_. As is, it is easy for the user to set these environment variables and forget in 6 months time what access key they have defined and where it is used. Or if any other package used the same environment variables without a prefix, you then are unable to set both defaults.
In my case, I have set as ACLED_EMAIL_ADDRESS and ACLED_ACCESS_KEY and explicitly pass the arg to acted.api() so the defaults go unused. I think long term, although initially breaking, would be good to move the environment variables to explicitly be something like these.