Move non-JSON/YAML/XML Parsers to orbiter-parsers#104
Conversation
fritz-astronomer
commented
Nov 6, 2025
- refactor: move parsers out
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the codebase by removing non-JSON/YAML/XML parsers (specifically JIL and Python parsers) from the main package, likely moving them to a separate orbiter-parsers package. This separation improves modularity by isolating specialized parsing functionality.
Key Changes:
- Removed dependencies on
jilutilandast2jsonpackages - Removed
FileTypeJILandFileTypePythonclasses along with their parsing logic - Cleaned up related imports and helper functions
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| pyproject.toml | Removed jilutil and ast2json dependencies |
| orbiter/file_types.py | Removed JIL and Python parser imports, classes (FileTypeJIL, FileTypePython), and helper functions (parse_jil, unimplemented_dump) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #104 +/- ##
==========================================
+ Coverage 78.92% 82.24% +3.32%
==========================================
Files 35 51 +16
Lines 1362 1876 +514
==========================================
+ Hits 1075 1543 +468
- Misses 287 333 +46 ☔ View full report in Codecov by Sentry. |