search

statelyai / sky

MIT License
8 stars 1 forks source link

Make API key required in config #13

Closed davidkpiano closed 10 months ago

davidkpiano commented 10 months ago

To simplify the API and make it possible to run multiple machines from different projects, we shouldn't implicitly grab the API key from .env, but instead do what other APIs do, which is require them directly.

We should direct users to put these API keys in their environment variables file and then read it in via process.env.ANY_API_KEY_VAR.

mellson commented 10 months ago

... and make it possible to run multiple machines from different projects

We need to update the xstate sky cmd for this to work properly. I've created STA-5554 for this work.

mellson commented 10 months ago

@davidkpiano Instead of you having to handle all our comments here I opened a new PR with this change here #14