-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add export options to the report and add testing suite #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds JSON and CSV export capabilities to the LintData reporting system, enabling programmatic use and spreadsheet analysis. The implementation includes comprehensive test coverage and maintains backward compatibility with existing text and HTML formats.
Key Changes
- Added JSON, CSV, and dictionary output formats to the
report()method - Implemented structured data parsing from warning strings with severity classification
- Added comprehensive test suite with 30+ tests covering all export formats and edge cases
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
tests/test_export.py |
New test file with comprehensive coverage for JSON, CSV, and dict exports including edge cases |
src/lintdata/accessor.py |
Extended report() method with new formats and added helper methods for parsing/formatting warnings |
README.md |
Updated roadmap to mark export options as complete |
CHANGELOG.md |
Documented new export features and helper methods |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Summary
Adds JSON and CSV export capabilities for programmatic use and spreadsheet analysis.
New Features
✅ JSON Export
✅ CSV Export
✅ Dictionary Output
return_dict=Truefor Python dictionaryUsage Examples
Documentation
Breaking Changes
None - fully backward compatible
Related
Part 3 of v0.7.0 development. Completes v0.7.0 feature set!