ci: removed isort

.pre-commit-config.yaml

@@ -46,12 +46,6 @@ repos:
                 lms/public/js/lib/.*
             )$
 
-
-  - repo: https://github.com/timothycrosley/isort
-    rev: 5.9.1
-    hooks:
-      - id: isort
-
   - repo: https://gitlab.com/pycqa/flake8
     rev: 3.9.2
     hooks:

lms/lms/doctype/invite_request/test_invite_request.py

@@ -5,7 +5,9 @@ import unittest
 import frappe
 
 from lms.lms.doctype.invite_request.invite_request import (
-				create_invite_request, update_invite)
+	create_invite_request,
+	update_invite,
+)
 
 
 class TestInviteRequest(unittest.TestCase):

lms/lms/doctype/lms_batch/lms_batch.py

@@ -5,8 +5,7 @@ import frappe
 from frappe import _
 from frappe.model.document import Document
 
-from lms.lms.doctype.lms_batch_membership.lms_batch_membership import \
+from lms.lms.doctype.lms_batch_membership.lms_batch_membership import create_membership
-				create_membership
 from lms.lms.utils import is_mentor
 
 

lms/lms/models.py

@@ -1,5 +1,6 @@
 """Handy module to make access to all doctypes from a single place.
 """
-from .doctype.lms_batch_membership.lms_batch_membership import \
+from .doctype.lms_batch_membership.lms_batch_membership import (
-				LMSBatchMembership as Membership
+	LMSBatchMembership as Membership,
+)
 from .doctype.lms_course.lms_course import LMSCourse as Course

lms/lms/utils.py

@@ -4,10 +4,17 @@ import string
 import frappe
 from frappe import _
 from frappe.desk.doctype.dashboard_chart.dashboard_chart import get_result
-from frappe.desk.doctype.notification_log.notification_log import \
+from frappe.desk.doctype.notification_log.notification_log import make_notification_logs
-				make_notification_logs
+from frappe.utils import (
-from frappe.utils import (add_months, cint, cstr, flt, fmt_money, format_date,
+	add_months,
-                          get_datetime, getdate)
+	cint,
+	cstr,
+	flt,
+	fmt_money,
+	format_date,
+	get_datetime,
+	getdate,
+)
 from frappe.utils.dateutils import get_period
 
 from lms.lms.md import find_macros, markdown_to_html

lms/www/batch/learn.py

@@ -2,8 +2,7 @@ import frappe
 from frappe import _
 from frappe.utils import cstr, flt
 
-from lms.lms.utils import (get_lesson_url, has_course_moderator_role,
+from lms.lms.utils import get_lesson_url, has_course_moderator_role, is_instructor
-                           is_instructor)
 from lms.www.utils import get_common_context, redirect_to_lesson
 
 

lms/www/courses/course.py

@@ -1,10 +1,15 @@
 import frappe
 from frappe import _
 
-from lms.lms.utils import (can_create_courses, get_evaluation_details,
+from lms.lms.utils import (
-                           get_membership, has_course_moderator_role,
+	can_create_courses,
-                           is_certified, is_instructor,
+	get_evaluation_details,
-                           redirect_to_courses_list)
+	get_membership,
+	has_course_moderator_role,
+	is_certified,
+	is_instructor,
+	redirect_to_courses_list,
+)
 
 
 def get_context(context):

lms/www/courses/index.py

@@ -1,8 +1,12 @@
 import frappe
 from frappe import _
 
-from lms.lms.utils import (can_create_courses, check_profile_restriction,
+from lms.lms.utils import (
-                           get_restriction_details, has_course_moderator_role)
+	can_create_courses,
+	check_profile_restriction,
+	get_restriction_details,
+	has_course_moderator_role,
+)
 
 
 def get_context(context):

lms/www/utils.py

@@ -1,7 +1,6 @@
 import frappe
 
-from lms.lms.utils import (get_batch, get_lesson_url, get_lessons,
+from lms.lms.utils import get_lesson_url, get_lessons, get_membership
-                           get_membership, slugify)
 
 
 def get_common_context(context):