Skip to content

Add type hints to parsers#993

Draft
GlassOfWhiskey wants to merge 9 commits intomainfrom
typed-python-parsers
Draft

Add type hints to parsers#993
GlassOfWhiskey wants to merge 9 commits intomainfrom
typed-python-parsers

Conversation

@GlassOfWhiskey
Copy link
Copy Markdown
Contributor

This commit enables type hints within Python generated parsers. by refactoring how the codegen logic handles instance types. Results are fully-typed Python objects ready to be compiled with mypyc.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 19, 2025

Codecov Report

❌ Patch coverage is 98.52941% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 83.32%. Comparing base (734fbc0) to head (67cd201).

Files with missing lines Patch % Lines
schema_salad/python_codegen.py 98.50% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #993      +/-   ##
==========================================
+ Coverage   83.23%   83.32%   +0.08%     
==========================================
  Files          22       22              
  Lines        5023     5042      +19     
  Branches     1195     1199       +4     
==========================================
+ Hits         4181     4201      +20     
+ Misses        559      558       -1     
  Partials      283      283              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@GlassOfWhiskey GlassOfWhiskey force-pushed the typed-python-parsers branch 2 times, most recently from 33bb828 to a8947d0 Compare December 20, 2025 21:08
@GlassOfWhiskey GlassOfWhiskey force-pushed the typed-python-parsers branch 3 times, most recently from 4b2b495 to 756ddb8 Compare April 4, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants