-
Notifications
You must be signed in to change notification settings - Fork 356
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
fix initialization bug for kwargs in function #827
Conversation
DCO Assistant Lite bot All contributors have signed the DCO ✍️ ✅ |
I have read the DCO Document and I hereby sign the DCO |
recheck |
thank you! will take a look |
This is a confirmed issue introduced by #711, when attempting to account for restricted parameter names. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we get a test class added that implements the example in the description?
Looks like the commit messages are missing the sign-off please adjust to ensure the branch is compliant with the DCO. |
3292592
to
dfef1a1
Compare
Signed-off-by: Soumili Nandi <[email protected]>
1c950d8
to
ac1966f
Compare
Generator.function
does not defineself.kwargs
when we don't pass that argument progamatically infunction.Single
.To reproduce this error, invoke garak's cli like below
Offered fix resolves this issue.