Code refactoring

Author: BobsProgrammingAcademy

config/settings.py

@@ -114,7 +114,7 @@
 STATIC_ROOT = os.path.join(BASE_DIR, 'staticfiles')
 
 # Location where we will store our static files
-STATICFILES_DIRS = [ os.path.join(BASE_DIR, 'static') ]
+STATICFILES_DIRS = [os.path.join(BASE_DIR, 'static')]
 
 MEDIA_URL = '/media/'
 MEDIA_ROOT = os.path.join(BASE_DIR, 'media')

portfolio/models.py

@@ -21,22 +21,22 @@ def __str__(self):
 class About(models.Model):
   title = models.CharField(max_length=50)
   description = models.CharField(max_length=250)
-  icon = models.CharField(max_length=30) # Material Icon name
-  
+  icon = models.CharField(max_length=30)  # Material Icon name
+
   class Meta:
     verbose_name = 'About'
     verbose_name_plural = 'About'
-    
+
   def __str__(self):
     return self.title
 
 
 class Tag(models.Model):
   name = models.CharField(max_length=30)
-  
+
   def __str__(self):
     return self.name
-    
+
 
 class Project(models.Model):
   name = models.CharField(max_length=100)
@@ -45,13 +45,13 @@ class Project(models.Model):
   image = models.ImageField()
   tags = models.ManyToManyField(Tag)
 
-  # override the save method and 
-  # use the Image class of the PIL package 
+  # override the save method and
+  # use the Image class of the PIL package
   # to convert it to JPEG
   def save(self, *args, **kwargs):
     if self.image:
       filename = "%s.jpg" % self.image.name.split('.')[0]
-      
+
       image = Image.open(self.image)
       # for PNG images discard the alpha channel and fill it with some color
       if image.mode in ('RGBA', 'LA'):
@@ -60,7 +60,7 @@ def save(self, *args, **kwargs):
         image = background
         image_io = BytesIO()
         image.save(image_io, format='JPEG', quality=100)
-                
+
         # change the image field value to be the newly modified image value
         self.image.save(filename, ContentFile(image_io.getvalue()), save=False)
     super(Project, self).save(*args, **kwargs)
@@ -73,18 +73,18 @@ class Contact(models.Model):
   address = models.CharField(max_length=250)
   email = models.CharField(max_length=150)
   phone = models.CharField(max_length=20)
-  
+
   class Meta:
     verbose_name = 'Contact'
     verbose_name_plural = 'Contact'
-    
+
   def __str__(self):
     return self.email
 
 
 class Footer(models.Model):
   copyright = models.CharField(max_length=200)
-  
+
   class Meta:
     verbose_name = 'Footer'
     verbose_name_plural = 'Footer'

portfolio/tests.py

@@ -8,10 +8,10 @@ def setUp(self):
         self.hero = Hero.objects.create(
             title='Lorem ipsum dolor sit amet',
             subtitle='Sed tincidunt quis odio id molestie',
-            description='Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed tincidunt quis odio id molestie.',
+            description='Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed tincidunt quis odio id.',
             image=tempfile.NamedTemporaryFile(suffix='.jpg').name
         )
 
     def test_hero_model(self):
         data = self.hero
-        self.assertIsInstance(data, Hero)
+        self.assertIsInstance(data, Hero)

portfolio/urls.py

@@ -3,4 +3,4 @@
 
 urlpatterns = [
   path('', IndexPageView.as_view()),
-]
+]

portfolio/views.py

@@ -1,9 +1,10 @@
 from django.views.generic.base import TemplateView
 from .models import Hero, About, Project, Contact, Footer
 
+
 class IndexPageView(TemplateView):
   template_name = 'index.html'
-  
+
   def get_context_data(self, **kwargs):
     context = super().get_context_data(**kwargs)
     context['hero_data'] = Hero.objects.all()

setup.cfg

@@ -0,0 +1,4 @@
+[flake8]
+exclude = .git,*migrations*,*venv*
+max-line-length = 119
+indent-size = 2