-
-
Notifications
You must be signed in to change notification settings - Fork 483
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
Update utils.py with adding a return of function bulk_update_with_history #1206
Conversation
keep the original returns of bulk_update
for more information, see https://pre-commit.ci
Codecov Report
@@ Coverage Diff @@
## master #1206 +/- ##
=======================================
Coverage 97.29% 97.30%
=======================================
Files 23 23
Lines 1259 1260 +1
Branches 204 204
=======================================
+ Hits 1225 1226 +1
Misses 16 16
Partials 18 18
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
Great! 😄
Some minor changes that would be nice:
Co-authored-by: Anders <6058745+ddabble@users.noreply.github.com>
fix suggestion
Thanks. |
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.
Thank you! 😊
Django version above 4.0 returns the rows_updated for function bulk_update.
Description
Related Issue
Motivation and Context
keep the same feature with the Django version above 4.0
How Has This Been Tested?
with the testcase
Screenshots (if appropriate):
Types of changes
Checklist:
pre-commit run
command to format and lint.AUTHORS.rst
CHANGES.rst