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

fix: Fixing the issue with the capturing of env variables #13

Merged
merged 2 commits into from
Sep 16, 2023

Conversation

noble-varghese
Copy link
Contributor

@noble-varghese noble-varghese commented Sep 16, 2023

Description

  • The base_url and api_key env were not picked up correctly from the env. This was more evident in the case of a notebook example and not so visible in the running of scripts from terminal.
  • Also moved the variables to global constants.

@noble-varghese noble-varghese merged commit e9db918 into main Sep 16, 2023
1 check failed
@noble-varghese noble-varghese deleted the noble-varghese/fix-env-fetch branch September 16, 2023 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant