Remove unused basic attribute on trigger build

Author: humitos

readthedocs/core/utils/__init__.py

@@ -75,7 +75,7 @@ def cname_to_slug(host):
     return slug
 
 
-def trigger_build(project, version=None, record=True, force=False, basic=False):
+def trigger_build(project, version=None, record=True, force=False):
     """
     Trigger build for project and version.
 
@@ -97,7 +97,6 @@ def trigger_build(project, version=None, record=True, force=False, basic=False):
         version_pk=version.pk,
         record=record,
         force=force,
-        basic=basic,
     )
 
     build = None

readthedocs/projects/views/private.py

@@ -231,10 +231,9 @@ def done(self, form_list, **kwargs):
         for field, value in list(form_data.items()):
             if field in extra_fields:
                 setattr(project, field, value)
-        basic_only = True
         project.save()
         project_import.send(sender=project, request=self.request)
-        trigger_build(project, basic=basic_only)
+        trigger_build(project)
         return HttpResponseRedirect(
             reverse('projects_detail', args=[project.slug]))
 
@@ -271,7 +270,7 @@ def get(self, request, *args, **kwargs):
             if form.is_valid():
                 project = form.save()
                 project.save()
-                trigger_build(project, basic=True)
+                trigger_build(project)
                 messages.success(
                     request, _('Your demo project is currently being imported'))
             else:

readthedocs/rtd_tests/tests/test_core_utils.py

@@ -30,7 +30,6 @@ def test_trigger_build_time_limit(self, update_docs):
                 kwargs={
                     'pk': self.project.id,
                     'force': False,
-                    'basic': False,
                     'record': True,
                     'build_pk': mock.ANY,
                     'version_pk': self.version.id
@@ -51,7 +50,6 @@ def test_trigger_build_invalid_time_limit(self, update_docs):
                 kwargs={
                     'pk': self.project.id,
                     'force': False,
-                    'basic': False,
                     'record': True,
                     'build_pk': mock.ANY,
                     'version_pk': self.version.id
@@ -72,7 +70,6 @@ def test_trigger_build_rounded_time_limit(self, update_docs):
                 kwargs={
                     'pk': self.project.id,
                     'force': False,
-                    'basic': False,
                     'record': True,
                     'build_pk': mock.ANY,
                     'version_pk': self.version.id