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.

Sign up for GitHub

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

Jump to bottom

Consider bailing out early when Pinecone variables are unset #32

Open
zackproser opened this issue Jan 8, 2024 · 0 comments
Open

Consider bailing out early when Pinecone variables are unset #32

zackproser opened this issue Jan 8, 2024 · 0 comments

Comments

@zackproser
Copy link
Contributor

Currently, it's possible to start the app without the two required environment variables being set:

  • PINECONE_API_KEY
  • PINECONE_ENVIRONMENT

which might be confusing for users, because the app will appear to work but there will be stack trace in the console.

Perhaps we should consider bailing out early, i.e., not starting up successfully if either of these variables are not set.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@zackproser