Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plans with positional only args cannot be run #780

Open
tpoliaw opened this issue Jan 13, 2025 · 1 comment
Open

Plans with positional only args cannot be run #780

tpoliaw opened this issue Jan 13, 2025 · 1 comment
Labels
bug Something isn't working documentation Improvements or additions to documentation

Comments

@tpoliaw
Copy link
Contributor

tpoliaw commented Jan 13, 2025

Given a plan such as

def demo(foo: int, /, bar: int) -> MsgGenerator:
    yield from ()

due to the method of wrapping all arguments into a dict and passing as demo(**args), it doesn't appear to be possible to run this plan.

This might not be an issue but if it's not intended to be supported, it would be good to document that somewhere and maybe warn when the plans are being read/registered.

@tpoliaw tpoliaw added bug Something isn't working documentation Improvements or additions to documentation labels Jan 13, 2025
@callumforrester
Copy link
Contributor

Thanks, no strong opinion on whether to support or just document

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants