8000 delete_orphans independent from blank by vchrisb · Pull Request #225 · codingjoe/django-stdimage · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

delete_orphans independent from blank #225

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

Merged
merged 3 commits into from
Mar 26, 2021
Merged

Conversation

vchrisb
Copy link
@vchrisb vchrisb commented Mar 24, 2021

fix for #224

@codecov
Copy link
codecov bot commented Mar 24, 2021

Codecov Report

Merging #225 (84d5cc9) into master (a99cb52) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #225   +/-   ##
=======================================
  Coverage   96.25%   96.25%           
=======================================
  Files           5        5           
  Lines         267      267           
=======================================
  Hits          257      257           
  Misses         10       10           
Impacted Files Coverage Δ
stdimage/models.py 94.97% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a99cb52...84d5cc9. Read the comment docs.

vchrisb added 2 commits March 25, 2021 16:17
* added test `test_pre_save_delete_callback_update`
* fixed and enhanced test `test_pre_save_delete_callback_new`
@codingjoe codingjoe merged commit e981a6d into codingjoe:master Mar 26, 2021
@codingjoe
Copy link
Owner

Thanks @vchrisb, great contribution. It's released in 5.2.1 🎉

Sign up for free to join this conversation on GitHub. Alr 59CE eady have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0