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

Updated env variable handling in 05-semantic-kernel.ipynb using dotenv #70

Closed

Chanchal-D wants to merge 1 commit into microsoft:main from Chanchal-D:main

Contributor

Chanchal-D commented Feb 24, 2025

Changes Made

Replaced os.environ[...] with dotenv in 05-semantic-kernel.ipynb
Added load_dotenv() to load variables from a .env file
Ensured compatibility and tested variable loading in Jupyter Notebook

Why This Change?

Improves flexibility by allowing environment variables to be loaded from .env
No need to manually set system environment variables

Testing Done

Verified environment variable loading in Jupyter Notebook
Ensured no unintended changes to outputs


          Update 05-semantic-kernel.ipynb

54dd415

github-actions bot commented Feb 24, 2025

👋 Thanks for contributing @Chanchal-D! We will review the pull request and get back to you soon.

github-actions bot added the needs-review label

github-actions bot assigned koreyspace

github-actions bot requested a review from koreyspace

February 24, 2025 20:26

Contributor Author

Chanchal-D commented Feb 24, 2025

@microsoft-github-policy-service agree

Chanchal-D closed this

Contributor Author

Chanchal-D commented Feb 24, 2025

This pr contains some errors - please do not merge.

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

Labels