Skip to content

Conversation

@allenporter
Copy link
Contributor

Document things like uv, pytest, where test data json files are. This can be improved (e.g. explaining how to test, use tooling, etc) but documenting what we have for now.

Copilot AI review requested due to automatic review settings January 6, 2026 15:43
Copy link
Contributor

Copilot AI left a comment

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 adds a comprehensive CONTRIBUTING.md file to document the contribution process for the python-roborock project. The document covers setup instructions using uv, development workflow, testing, and guidelines for adding new devices or features.

  • Establishes development environment setup with uv and pre-commit hooks
  • Documents code quality tools (Ruff, Mypy, Codespell) and testing with pytest
  • Provides clear guidelines for conventional commits and instructions for adding new device support

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


1. **Update Device Info**: Use the CLI to discover and fetch device features.
```bash
roborock get-device-info
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

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

The CLI command should use hyphens instead of underscores. Click (the CLI framework used by this project) automatically converts underscores in Python function names to hyphens in the command-line interface. Since the function is named 'get_device_info', the actual command should be 'roborock get-device-info'.

Copilot uses AI. Check for mistakes.
@Lash-L
Copy link
Collaborator

Lash-L commented Jan 6, 2026

Can you add info for activating the venv before pre commit install?

@allenporter
Copy link
Contributor Author

Can you add info for activating the venv before pre commit install?

Done.

@allenporter allenporter merged commit 881b7d6 into Python-roborock:main Jan 7, 2026
7 checks passed
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.

2 participants