Add dataset schema command - #126
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (6)
Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe pull request adds ChangesDataset schema
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Feature · Severity of issue fixed: Medium Sequence Diagram(s)sequenceDiagram
participant CLI as pb CLI
participant Command as SchemaDatasetCmd
participant API as Dataset schema API
CLI->>Command: Run schema command
Command->>API: GET dataset schema
API-->>Command: Schema response
Command-->>CLI: Write JSON or rendered fields
Merge Risk: ⚪ Minimal · up to The schema command appears consistent with the requested text and JSON behavior and read-only catalog integration. No actionable merge-blocking issue is established; normal test checks should pass before merging. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 5 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@CodeRabbit review |
|
Closes #120
pb dataset schema <dataset>to show field names and types.Summary by CodeRabbit
pb dataset schema <dataset>to display dataset field names and types in text or JSON format.