Skip to content

fix(load_dotenv): remove load_dotenv()#68

Merged
giuliastf merged 1 commit into
mainfrom
fix/load_dotenv
Sep 18, 2025
Merged

fix(load_dotenv): remove load_dotenv()#68
giuliastf merged 1 commit into
mainfrom
fix/load_dotenv

Conversation

@giuliastf

@giuliastf giuliastf commented Sep 17, 2025

Copy link
Copy Markdown
Collaborator

This pull request removes the use of the python-dotenv library and all calls to load_dotenv() from both sample scripts and the CLI utility code.

Development Package

  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath-llamaindex==0.0.36.dev1000680211",

  # Any version from PR
  "uipath-llamaindex>=0.0.36.dev1000680000,<0.0.36.dev1000690000"
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath-llamaindex = { index = "testpypi" }

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the python-dotenv library dependency by eliminating all imports and calls to load_dotenv() across the codebase. The changes affect both CLI utilities and sample scripts.

  • Removed load_dotenv() import and call from CLI runner module
  • Removed environment file loading functionality from the configuration utility
  • Cleaned up sample scripts by removing dotenv imports and initialization calls

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/uipath_llamaindex/_cli/cli_run.py Removed dotenv import and load_dotenv() call
src/uipath_llamaindex/_cli/_utils/_config.py Removed dotenv import and env file loading logic
samples/quickstart-llamacloud/main.py Removed dotenv import and load_dotenv() call
samples/multi-agent/main.py Removed dotenv import and load_dotenv() call
samples/action-center-hitl-agent/main.py Removed dotenv import and load_dotenv() call

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/uipath_llamaindex/_cli/_utils/_config.py
@giuliastf giuliastf force-pushed the fix/load_dotenv branch 2 times, most recently from e41253f to 4fdcc47 Compare September 17, 2025 15:31
@giuliastf giuliastf self-assigned this Sep 17, 2025
@giuliastf giuliastf added the build:dev Create a dev build from the pr label Sep 18, 2025
@giuliastf giuliastf merged commit 3585f6f into main Sep 18, 2025
21 checks passed
@cristipufu cristipufu deleted the fix/load_dotenv branch November 30, 2025 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build:dev Create a dev build from the pr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants