Fixed the issue with insert returning query by anandology · Pull Request #649 · webpy/webpy
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 1.3k
Merged
iredmail merged 2 commits intowebpy:masterfrom
Jun 24, 2020Merged
Fixed the issue with insert returning query#649
iredmail merged 2 commits intowebpy:masterfrom
Fixed the issue with insert returning query#649
iredmail merged 2 commits intowebpy:masterfrom
Conversation
The insert ... returning ... query was not getting commited to the db due to a recent change. Issue webpy#648
anandology
requested a review
from iredmail
anandology
force-pushed
the
648-insert-returning
branch
from
1f60363 to
1fea0d9
Compare
iredmail previously approved these changes Jun 24, 2020
Copy link Copy Markdown
Contributor
iredmail
left a comment
iredmail
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Copy link Copy Markdown
Member Author
anandology
commented
Jun 24, 2020
anandology commented
Jun 24, 2020These black issues are really frustrating.
anandology
dismissed
iredmail’s stale review
via
22226c5
anandology
force-pushed
the
648-insert-returning
branch
from
1fea0d9 to
22226c5
Compare
This test was getting blocked when tried without the fix in the previous commit. Issue webpy#648
anandology
force-pushed
the
648-insert-returning
branch
from
22226c5 to
0067edd
Compare
Copy link Copy Markdown
Member Author
anandology
commented
Jun 24, 2020
anandology commented
Jun 24, 2020Fixed the black errors.
iredmail approved these changes Jun 24, 2020
iredmail
merged commit
b8e6eba
into
webpy:master
anandology
deleted the
648-insert-returning
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment