Fixed pipenv compatibility issue

[jawad-khan]

Pull Request

Related issue

Fixes #1193

What does this PR do?

• This PR solves known version mismatch between pipenv and its vendored pipdeptree.

PR checklist

Please check if your PR fulfills the following requirements:

• Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
• Have you read the contributing guidelines?
• Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Summary by CodeRabbit

• Chores
  • Updated the dependency installation process in continuous integration and deployment workflows for consistency across test and release pipelines.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Replaced pipx-based pipenv installation with direct pip installation across four GitHub Actions workflows (documentation, pre-release tests, PyPI publishing, and tests). This standardizes the pipenv setup method and addresses the failing documentation build workflow.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflows .github/workflows/documentation.yml, .github/workflows/pre-release-tests.yml, .github/workflows/pypi-publish.yml, .github/workflows/tests.yml	Replaced pipx install pipenv with pip install pipenv across all workflow installation steps. This changes the pipenv installation mechanism from using pipx as an intermediary to direct pip installation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A pip, a pipx, away they go,
Direct and swift, like morning snow,
No middleman in our workflow's dance,
Documentation now gets its chance! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The PR changes pipenv installation method across all workflows to fix issue #1193 (failing documentation build), directly addressing the stated objective.
Out of Scope Changes check	✅ Passed	All changes are focused on replacing pipx-based pipenv installation with direct pip installation across four workflow files, which is directly related to fixing the pipenv compatibility issue.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Fixed pipenv compatibility issue' accurately describes the main change across all modified files: replacing pipx-based pipenv installation with direct pip installation to resolve a pipenv compatibility problem.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

.github/workflows/tests.yml (1)

47-48: Inconsistent: pylint job still uses pipx install pipenv.

This job was not updated and still uses pipx install pipenv, which contradicts the PR's objective to fix pipenv compatibility issues across all workflows.

Proposed fix

      - name: Install pipenv
-        run: pipx install pipenv
+        run: pip install pipenv

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6003393 and 0da1c2a.

📒 Files selected for processing (4)

• .github/workflows/documentation.yml
• .github/workflows/pre-release-tests.yml
• .github/workflows/pypi-publish.yml
• .github/workflows/tests.yml

🔇 Additional comments (7)

.github/workflows/tests.yml (4)

29-29: LGTM!

The pipenv installation method is correctly updated to use pip install pipenv.

---

65-65: LGTM!

Consistent with the other updated jobs.

---

82-82: LGTM!

Consistent with the other updated jobs.

---

99-99: LGTM!

Consistent with the other updated jobs.

.github/workflows/pypi-publish.yml (1)

20-21: LGTM!

The pipenv installation method is correctly updated to use pip install pipenv, consistent with the PR's objective.

.github/workflows/documentation.yml (1)

39-41: LGTM!

This change directly addresses the failing documentation build referenced in issue #1193. The pipenv installation method is correctly updated.

.github/workflows/pre-release-tests.yml (1)

26-27: LGTM!

The pipenv installation method is correctly updated to use pip install pipenv, consistent with the other workflow files.

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

[Strift]

hey @jawad-khan, thanks for the fix :)