diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index c79c736f..33ad3b85 100644 --- a/.pre-commit-config.yaml +++ b/.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: diff --git a/lms/lms/doctype/invite_request/test_invite_request.py b/lms/lms/doctype/invite_request/test_invite_request.py index 7e91567d..b9cac5d3 100644 --- a/lms/lms/doctype/invite_request/test_invite_request.py +++ b/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): diff --git a/lms/lms/doctype/lms_batch/lms_batch.py b/lms/lms/doctype/lms_batch/lms_batch.py index 8fdb4182..b2d13a79 100644 --- a/lms/lms/doctype/lms_batch/lms_batch.py +++ b/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 \ - create_membership +from lms.lms.doctype.lms_batch_membership.lms_batch_membership import create_membership from lms.lms.utils import is_mentor diff --git a/lms/lms/models.py b/lms/lms/models.py index 82cece18..ec2cd230 100644 --- a/lms/lms/models.py +++ b/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 \ - LMSBatchMembership as Membership +from .doctype.lms_batch_membership.lms_batch_membership import ( + LMSBatchMembership as Membership, +) from .doctype.lms_course.lms_course import LMSCourse as Course diff --git a/lms/lms/utils.py b/lms/lms/utils.py index e10e1010..b4480f50 100644 --- a/lms/lms/utils.py +++ b/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 \ - make_notification_logs -from frappe.utils import (add_months, cint, cstr, flt, fmt_money, format_date, - get_datetime, getdate) +from frappe.desk.doctype.notification_log.notification_log import make_notification_logs +from frappe.utils import ( + add_months, + 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 diff --git a/lms/www/batch/learn.py b/lms/www/batch/learn.py index 85fd2c5e..a076cdf8 100644 --- a/lms/www/batch/learn.py +++ b/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, - is_instructor) +from lms.lms.utils import get_lesson_url, has_course_moderator_role, is_instructor from lms.www.utils import get_common_context, redirect_to_lesson diff --git a/lms/www/courses/course.py b/lms/www/courses/course.py index 93921071..d9404ed5 100644 --- a/lms/www/courses/course.py +++ b/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, - get_membership, has_course_moderator_role, - is_certified, is_instructor, - redirect_to_courses_list) +from lms.lms.utils import ( + can_create_courses, + get_evaluation_details, + get_membership, + has_course_moderator_role, + is_certified, + is_instructor, + redirect_to_courses_list, +) def get_context(context): diff --git a/lms/www/courses/index.py b/lms/www/courses/index.py index e1f422bb..68e6acf3 100644 --- a/lms/www/courses/index.py +++ b/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, - get_restriction_details, has_course_moderator_role) +from lms.lms.utils import ( + can_create_courses, + check_profile_restriction, + get_restriction_details, + has_course_moderator_role, +) def get_context(context): diff --git a/lms/www/utils.py b/lms/www/utils.py index 1ce52227..74bc6f88 100644 --- a/lms/www/utils.py +++ b/lms/www/utils.py @@ -1,7 +1,6 @@ import frappe -from lms.lms.utils import (get_batch, get_lesson_url, get_lessons, - get_membership, slugify) +from lms.lms.utils import get_lesson_url, get_lessons, get_membership def get_common_context(context):