There's actually no harm keeping this around as it makes the code. Remove the TODO, replacing it with a note on why the function exists.
Signed-off-by: Stephen Finucane <[email protected]> --- patchwork/models.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/patchwork/models.py b/patchwork/models.py index f6a2f26..b18213d 100644 --- a/patchwork/models.py +++ b/patchwork/models.py @@ -296,7 +296,8 @@ class Submission(EmailMixin, models.Model): # patchwork metadata def refresh_tag_counts(self): - pass # TODO(sfinucan) Once this is only called for patches, remove + # This is subclassed on 'Patch' to do something useful + pass def is_editable(self, user): return False -- 2.7.4 _______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
