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

Fix wrapping context on named arguments. #111

Merged
merged 2 commits into from
Aug 27, 2024

Conversation

foobarna
Copy link
Contributor

No description provided.

@davecoates
Copy link

davecoates commented Aug 7, 2024

I encountered this when trying to use procrastinate - deferring a task would hit an error about mixed arguments. This patch seems to work well. The error in question was:

django.db.utils.ProgrammingError: positional and named placeholders cannot be mixed

@wesleykendall
Copy link
Member

thank you! I will see if I can write a test case for this for additional measure before releasing

@wesleykendall wesleykendall merged commit 1331514 into Opus10:main Aug 27, 2024
0 of 4 checks passed
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