skrawcz commented on PR #1402:
URL: https://github.com/apache/hamilton/pull/1402#issuecomment-3388454435
@harshith1118 can you fix the pre-commit errors please:
```
ruff.....................................................................Failed
- hook id: ruff
- exit code: 1
- files were modified by this hook
hamilton/cli/logic.py:349:9: B904 Within an `except` clause, raise
exceptions with `raise ... from err` or `raise ... from None` to distinguish
them from errors in exception handling
|
347 | except ImportError:
348 | # Provide a helpful error message if tomli is not available
349 | raise ImportError(
| _________^
350 | | "tomli is required to read TOML files. "
351 | | "Install it with `pip install tomli` or `pip install
sf-hamilton[cli]` which includes TOML support."
352 | | )
| |_________^ B904
353 |
354 | with open(file_path, "rb") as f:
|
Found 15 errors (14 fixed, 1 remaining).
ruff-format..............................................................Passed
trim trailing
whitespace.................................................Passed
fix end of
files.........................................................Failed
- hook id: end-of-file-fixer
- exit code: 1
- files were modified by this hook
Fixing tests/cli/resources/test_context.toml
Fixing tests/cli/resources/test_tool_hamilton.toml
fix
requirements.txt.....................................................Passed
check python
ast.........................................................Passed
validate example
notebooks...............................................Passed
```
you can run this by installing the pre-commit hook and then running it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
