Skip to content

ci: install dependencies in the test environment when testing with lowest direct dependencies and Haystack main#2418

Merged
anakin87 merged 5 commits intomainfrom
install-deps-on-test-environment
Oct 21, 2025
Merged

ci: install dependencies in the test environment when testing with lowest direct dependencies and Haystack main#2418
anakin87 merged 5 commits intomainfrom
install-deps-on-test-environment

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Oct 21, 2025

Related Issues

Our workflow step to test an integration with lowest direct dependencies is not functioning correctly.

Tests are being executed from the test Hatch environment, but dependencies are installed in the default environment.

Proposed Changes:

  • modify the test workflow to install dependencies in the test environment
  • pin some dependencies to fix failures

How did you test it?

CI; tests failures are related to ToolsType.

Notes for the reviewer

Not sure why this workflow step sometimes worked.

Checklist

@anakin87 anakin87 self-assigned this Oct 21, 2025
@anakin87 anakin87 changed the title CI: install deps on test environment ci: install dependencies in the test environment when testing with lowest direct dependencies and Haystack main Oct 21, 2025
@anakin87 anakin87 marked this pull request as ready for review October 21, 2025 13:22
@anakin87 anakin87 requested a review from a team as a code owner October 21, 2025 13:22
@anakin87 anakin87 requested review from mpangrazzi and sjrl and removed request for a team and mpangrazzi October 21, 2025 13:22
run: |
hatch run uv pip compile pyproject.toml --resolution lowest-direct --output-file requirements_lowest_direct.txt
hatch run uv pip install -r requirements_lowest_direct.txt
hatch -e test env run -- uv pip install -r requirements_lowest_direct.txt
Copy link
Copy Markdown
Member Author

@anakin87 anakin87 Oct 21, 2025

Choose a reason for hiding this comment

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

I initially tried hatch run -e test uv pip install -r requirements_lowest_direct.txt
but Hatch fails to interpret correctly the command.

Copy link
Copy Markdown
Contributor

@sjrl sjrl left a comment

Choose a reason for hiding this comment

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

Looks good!

@anakin87 anakin87 merged commit 8e880d3 into main Oct 21, 2025
184 of 194 checks passed
@anakin87 anakin87 deleted the install-deps-on-test-environment branch October 21, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment