Skip to content

Remove workaround from select_subclasess()#524

Merged
hramezani merged 1 commit intojazzband:masterfrom
adamchainz:remove_workaround
Aug 17, 2022
Merged

Remove workaround from select_subclasess()#524
hramezani merged 1 commit intojazzband:masterfrom
adamchainz:remove_workaround

Conversation

@adamchainz
Copy link
Copy Markdown
Contributor

Problem

The workaround was for an issue that was fixed in Django 1.7: https://code.djangoproject.com/ticket/16855 / django/django@349c12d .

Solution

Remove workaround.

Commandments

  • Write PEP8 compliant code.
  • Cover it with tests.
  • Update CHANGES.rst file to describe the changes, and quote according issue with GH-<issue_number>.
  • Pay attention to backward compatibility, or if it breaks it, explain why.
  • Update documentation (if relevant).

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 4, 2022

Codecov Report

Merging #524 (5382b92) into master (02bd67c) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #524      +/-   ##
==========================================
- Coverage   95.15%   95.12%   -0.03%     
==========================================
  Files           6        6              
  Lines         825      820       -5     
==========================================
- Hits          785      780       -5     
  Misses         40       40              
Impacted Files Coverage Δ
model_utils/managers.py 98.08% <ø> (-0.05%) ⬇️

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 02bd67c...5382b92. Read the comment docs.

@hramezani hramezani merged commit d699745 into jazzband:master Aug 17, 2022
@adamchainz adamchainz deleted the remove_workaround branch August 17, 2022 21:36
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.

2 participants