Skip to content

ordeq: more informative error for nodes with kw-only args w/o default#516

Merged
nielsneerhoff merged 5 commits into
mainfrom
kwarg-only-wo-default
Dec 12, 2025
Merged

ordeq: more informative error for nodes with kw-only args w/o default#516
nielsneerhoff merged 5 commits into
mainfrom
kwarg-only-wo-default

Conversation

@nielsneerhoff
Copy link
Copy Markdown
Member

@nielsneerhoff nielsneerhoff commented Dec 11, 2025

Before this PR the error for nodes with keyword only args without default was too generic:

ValueError: Inputs invalid for function arguments of view 'func2' in module '__main__'

Now it clearly indicates what's wrong:

ValueError: Keyword-only argument 'a' of view 'func2' in module '__main__' must have a default

@nielsneerhoff nielsneerhoff added the fix Something isn't working label Dec 11, 2025
@nielsneerhoff nielsneerhoff enabled auto-merge (squash) December 11, 2025 17:29
@nielsneerhoff nielsneerhoff merged commit cb01b2f into main Dec 12, 2025
41 checks passed
@nielsneerhoff nielsneerhoff deleted the kwarg-only-wo-default branch December 12, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants