Skip to content

Conversation

@ryankert01
Copy link
Contributor

Purpose of PR

Related Issues or PRs

Changes Made

  • Bug fix
  • New feature
  • Refactoring
  • Documentation
  • Test
  • CI/CD pipeline
  • Other

Breaking Changes

  • Yes
  • No

Checklist

  • Added or updated unit tests for all changes
  • Added or updated documentation for all changes
  • Successfully built and ran all unit tests or manual tests locally
  • PR title follows "MAHOUT-XXX: Brief Description" format (if related to an issue)
  • Code follows ASF guidelines

Updated build instructions to include Python 3.11 environment setup.
@ryankert01 ryankert01 marked this pull request as ready for review December 8, 2025 02:56
@ryankert01 ryankert01 changed the base branch from main to dev-qdp December 8, 2025 02:56
@ryankert01
Copy link
Contributor Author

PTAL @rich7420

source .venv/bin/activate
```
```bash
uv sync --group dev
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I personally cannot use --extra, because of following(from chatgpt):
dependency-groups 是 Hatch/uv 新格式,但要用 uv sync --group dev

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's right!

@rich7420
Copy link
Contributor

rich7420 commented Dec 8, 2025

LGTM

@guan404ming guan404ming merged commit 7da8b90 into apache:dev-qdp Dec 8, 2025
4 checks passed
guan404ming pushed a commit to guan404ming/mahout that referenced this pull request Dec 11, 2025
Updated build instructions to include Python 3.11 environment setup.
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.

3 participants