Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| uv pip install -e ".[dev,test,docs,examples]" | ||
| git submodule update --init --recursive # fetch test data | ||
| pytest tests/ --cov | ||
| unittest tests/ --cov |
There was a problem hiding this comment.
Important: unittest tests/ --cov is not valid syntax. unittest is not a standalone CLI command, and it does not accept --cov. This was previously pytest tests/ --cov (the correct command) and was regressed in this PR.
| unittest tests/ --cov | |
| pytest tests/ --cov |
|
PR Review summary for this PR (inline comment also posted on CLAUDE.md line 16): The only important issue found is a regression in CLAUDE.md where pytest tests/ --cov was changed to unittest tests/ --cov. This is invalid: unittest is not a standalone CLI tool and does not accept --cov. The original pytest tests/ --cov should be restored. All other changes look correct: the html_meta front matter in user_guide.md is the proper MyST replacement for the old meta directive, removing -e from the links workflow install is fine since only doc-building is needed, adding .ruff_cache/ to .gitignore is correct, and the submodule bumps are routine. |
1 participant
No description provided.