Skip to content

Commit 1bbebd4

Browse files
fix: CI workflow improvements
- Fix installation test by using virtual environment instead of --system flag - Run pre-commit hooks to fix formatting issues (trailing whitespace, end-of-file)
1 parent 6a11151 commit 1bbebd4

12 files changed

Lines changed: 13270 additions & 24 deletions

.cursor/rules/linting-philosophy.mdc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,9 @@ Only **truly global style choices** that apply project-wide:
4242
- `PLR0913` - Argument count (specific functions)
4343

4444
## Benefits
45-
✅ **Strict main code**: Source files enforce high quality standards
46-
✅ **Appropriate flexibility**: Tests and examples have relaxed rules where needed
47-
✅ **Clear rationale**: Each ignore is documented with specific reasoning
45+
✅ **Strict main code**: Source files enforce high quality standards
46+
✅ **Appropriate flexibility**: Tests and examples have relaxed rules where needed
47+
✅ **Clear rationale**: Each ignore is documented with specific reasoning
4848
✅ **Maintainable**: File-specific concerns handled at file level, not globally
4949

5050
## Verification

.cursor/rules/testing-philosophy.mdc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ This project uses **behavioral (black-box) testing** instead of white-box testin
1818
- Test with realistic data scenarios
1919

2020
### ❌ DON'T: Test Internal Implementation
21-
- Avoid testing private methods directly
21+
- Avoid testing private methods directly
2222
- Don't test internal data structures
2323
- Don't mock internal implementation details
2424
- Don't test "how" the code works, test "what" it does

.github/workflows/ci.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -141,11 +141,14 @@ jobs:
141141
- name: Set up Python
142142
run: uv python install
143143

144+
- name: Create virtual environment
145+
run: uv venv
146+
144147
- name: Test wheel installation
145-
run: uv pip install --system dist/*.whl
148+
run: uv pip install dist/*.whl
146149

147150
- name: Test package import
148-
run: python -c "import clearml_mcp; print('✅ Package imported successfully')"
151+
run: uv run python -c "import clearml_mcp; print('✅ Package imported successfully')"
149152

150153
- name: Test CLI entry point
151-
run: clearml-mcp --help || echo "✅ CLI entry point exists (may require ClearML config)"
154+
run: uv run clearml-mcp --help || echo "✅ CLI entry point exists (may require ClearML config)"

.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,4 +18,4 @@ repos:
1818
- id: ruff-format
1919

2020
# Removed mypy from pre-commit due to dependency issues
21-
# Type checking is handled separately in CI
21+
# Type checking is handled separately in CI

.specstory/.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
# SpecStory explanation file
2+
/.what-is-this.md

.specstory/history/2025-07-12_19-35Z-update-goals-for-clearml-mcp-repo.md

Lines changed: 2282 additions & 0 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)