mirror of
https://github.com/OpenHands/OpenHands.git
synced 2025-12-26 05:48:36 +08:00
docs: Improve enterprise linting guidance in repo.md
- Add emphasis on --show-diff-on-failure flag importance - Add troubleshooting note for GitHub CI vs local linting differences - Ensures developers follow exact same commands as GitHub CI
This commit is contained in:
parent
33b9044ed3
commit
21702895a5
@ -114,7 +114,7 @@ PYTHONPATH=".:$PYTHONPATH" poetry run --project=enterprise pytest --forked -n au
|
||||
cd enterprise
|
||||
PYTHONPATH=".:$PYTHONPATH" poetry run pytest tests/unit/telemetry/ --confcutdir=tests/unit/telemetry
|
||||
|
||||
# Enterprise linting
|
||||
# Enterprise linting (IMPORTANT: use --show-diff-on-failure to match GitHub CI)
|
||||
poetry run pre-commit run --all-files --show-diff-on-failure --config ./dev_config/python/.pre-commit-config.yaml
|
||||
```
|
||||
|
||||
@ -191,6 +191,7 @@ Each integration follows a consistent pattern with service classes, storage mode
|
||||
- For frontend issues, ensure the main OpenHands frontend is built: `make build`
|
||||
- Check logs in the `logs/` directory for runtime issues
|
||||
- If tests fail with import errors, verify `PYTHONPATH=".:$PYTHONPATH"` is set
|
||||
- **If GitHub CI fails but local linting passes**: Always use `--show-diff-on-failure` flag to match CI behavior exactly
|
||||
|
||||
## Template for Github Pull Request
|
||||
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user