feat: add --json output flag to auditwheel show#685
feat: add --json output flag to auditwheel show#685
auditwheel show#685Conversation
mayeut
left a comment
mayeut
left a comment
There was a problem hiding this comment.
Not reviewed thoroughly yet but left a couple comments inlined in addition to the following more general comment.
Per #676 (comment), as the json output should be an "API", it should be versioned, and, ideally, we might want to have a json schema for this.
|
While this does not cover all functionality requested in #676, it does solve one of them so linking here. |
The docker SDK types `exec_run` as returning `(int | None, bytes | Iterator[bytes])`, but in non-streaming mode it always returns `(int, bytes)`. Add isinstance assertions to satisfy mypy. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Enable scripting by adding a --json flag that outputs structured JSON instead of human-readable text, including wheel tag, compatibility flags, versioned symbols, external libs, and policy upgrade info. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
| ) | ||
| except NonPlatformWheelError as e: | ||
| logger.info("%s", e.message) | ||
| if args.JSON: |
There was a problem hiding this comment.
I'm not sure if handling this special json case is important. Also, the case below where pure wheels are allowed does not emit a json. What's the preference?
There was a problem hiding this comment.
What's the preference?
If adding this, let's try to add it properly.
If pure wheels are allowed then we should not error but output a specific json output for pure wheels, e.g. the schema could have something like:
"oneOf": [
{ "$ref": "#/$defs/result_pure" },
{ "$ref": "#/$defs/result_platform" },
{ "$ref": "#/$defs/error" }
],
adding a "pure" boolean property that's always true for result_pure and always false for result_platform
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #685 +/- ##
==========================================
+ Coverage 95.24% 95.31% +0.06%
==========================================
Files 22 22
Lines 1830 1856 +26
Branches 343 350 +7
==========================================
+ Hits 1743 1769 +26
Misses 48 48
Partials 39 39 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
2 participants
Enable scripting by adding a --json flag that outputs structured JSON instead of human-readable text, including wheel tag, compatibility flags, versioned symbols, external libs, and policy upgrade info.