Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add a function to set API key in ENV #13

Merged
merged 7 commits into from
Feb 24, 2019
Merged

add a function to set API key in ENV #13

merged 7 commits into from
Feb 24, 2019

Conversation

gino244
Copy link

@gino244 gino244 commented Feb 24, 2019

This function make programmers using this package easy to start calling the package functions. When I tested the code in Usage on https://github.com/ellisvalentiner/AlphaVantage.jl, I faced an error below:

ERROR: KeyError: key "ALPHA_VANTAGE_API_KEY" not found

I had noticed to have to set the environment variable "ALPHA_VANTAGE_API_KEY" before calling functions when I saw the code. Using this function `set_apikey' will make them use the key only during running code. After finishing code, the key is gone, and make environment clean.

ellisvalentiner and others added 7 commits November 5, 2017 10:33
* Add tests for daily and daily adjusted

* Allow travis failures on nightly builds

* Fix wrong function names in API calls

* Add tests for weekly, monthly functions

* Fix typo

* More tests

* Begin documentation site (#4)

* Reorganize documentaiton

* Begin reorganization

* Code generation is amazing

* Major overhaul

* Update documentation

* Add ArgCheck to REQUIRE

* Fix missing parameter for digital currencies

* Fix deployment fail due to misspecified theme
Add Project.toml

Add Manifest.toml

Update REQUIRE

Replace Requests with HTTP
@ellisvalentiner ellisvalentiner self-assigned this Feb 24, 2019
@ellisvalentiner ellisvalentiner merged commit 929056c into ellisvalentiner:dev Feb 24, 2019
@ellisvalentiner
Copy link
Owner

Looks good to me. Thanks for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants