Skip to content

Add ref_template to json.pyi#1002

Merged
jcrist merged 1 commit intojcrist:mainfrom
sobolevn:patch-3
Apr 8, 2026
Merged

Add ref_template to json.pyi#1002
jcrist merged 1 commit intojcrist:mainfrom
sobolevn:patch-3

Conversation

@sobolevn
Copy link
Copy Markdown
Contributor

@sobolevn sobolevn commented Apr 8, 2026

Follow up to #983 Sorry, I've missed that json.pyi exists :(

Without this change mypy is not happy:

» mypy .
dmr/plugins/msgspec/schema.py:23: error: Unexpected keyword argument "ref_template" for "schema"  [call-arg]
.venv/lib/python3.13/site-packages/msgspec/json.pyi: note: "schema" defined here
Found 1 error in 1 file (checked 472 source files)

Follow up to jcrist#983
Sorry, I've missed that `json.pyi` exists :(

Without this change mypy is not happy:

```
» mypy .
dmr/plugins/msgspec/schema.py:23: error: Unexpected keyword argument "ref_template" for "schema"  [call-arg]
.venv/lib/python3.13/site-packages/msgspec/json.pyi: note: "schema" defined here
Found 1 error in 1 file (checked 472 source files)
```
@jcrist jcrist merged commit 4024c90 into jcrist:main Apr 8, 2026
22 checks passed
@ofek
Copy link
Copy Markdown
Collaborator

ofek commented Apr 8, 2026

I think we should cut a patch release for this.

@jcrist
Copy link
Copy Markdown
Owner

jcrist commented Apr 8, 2026

Lets hold off a few days, I have at least one more PR I want to get in and I suspect there may be others.

@ofek
Copy link
Copy Markdown
Collaborator

ofek commented Apr 10, 2026

@Siyet Starting right now I will be away for 2 weeks. Please feel free to merge and release fixes without our presence.

@Siyet Siyet mentioned this pull request Apr 12, 2026
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.

3 participants