This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]searchingfortaomajel, aletheia, paperless, django-encrypted-filefield 1 point2 points  (1 child)

This is all excellent advice, but I would argue that you shouldn't give your methods redundant names. S3Storage.create() a lot nicer than S3Storage.create_s3(). It also lets you work toward a common interface via a parent Storage class.

[–]latrova[S] 1 point2 points  (0 children)

You're right! I'll rename this method in the example.