Add pre-commit check

.editorconfig

@@ -0,0 +1,15 @@
+# Root editor config file
+root = true
+
+# Common settings
+[*]
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+charset = utf-8
+
+# python, js indentation settings
+[{*.py,*.js,*.vue,*.css,*.scss,*.html}]
+indent_style = tab
+indent_size = 4
+max_line_length = 99

.eslintrc

@@ -0,0 +1,155 @@
+{
+    "env": {
+        "browser": true,
+        "node": true,
+        "es6": true
+    },
+    "parserOptions": {
+        "ecmaVersion": 11,
+        "sourceType": "module"
+    },
+    "extends": "eslint:recommended",
+    "rules": {
+        "indent": [
+            "error",
+            "tab",
+            { "SwitchCase": 1 }
+        ],
+        "brace-style": [
+            "error",
+            "1tbs"
+        ],
+        "space-unary-ops": [
+            "error",
+            { "words": true }
+        ],
+        "linebreak-style": [
+            "error",
+            "unix"
+        ],
+        "quotes": [
+            "off"
+        ],
+        "semi": [
+            "warn",
+            "always"
+        ],
+        "camelcase": [
+            "off"
+        ],
+        "no-unused-vars": [
+            "warn"
+        ],
+        "no-redeclare": [
+            "warn"
+        ],
+        "no-console": [
+            "warn"
+        ],
+        "no-extra-boolean-cast": [
+            "off"
+        ],
+        "no-control-regex": [
+            "off"
+        ],
+        "space-before-blocks": "warn",
+        "keyword-spacing": "warn",
+        "comma-spacing": "warn",
+        "key-spacing": "warn",
+    },
+    "root": true,
+    "globals": {
+        "frappe": true,
+        "Vue": true,
+        "__": true,
+        "repl": true,
+        "Class": true,
+        "locals": true,
+        "cint": true,
+        "cstr": true,
+        "cur_frm": true,
+        "cur_dialog": true,
+        "cur_page": true,
+        "cur_list": true,
+        "cur_tree": true,
+        "msg_dialog": true,
+        "is_null": true,
+        "in_list": true,
+        "has_common": true,
+        "has_words": true,
+        "validate_email": true,
+        "validate_name": true,
+        "validate_phone": true,
+        "validate_url": true,
+        "get_number_format": true,
+        "format_number": true,
+        "format_currency": true,
+        "comment_when": true,
+        "open_url_post": true,
+        "toTitle": true,
+        "lstrip": true,
+        "rstrip": true,
+        "strip": true,
+        "strip_html": true,
+        "replace_all": true,
+        "flt": true,
+        "precision": true,
+        "CREATE": true,
+        "AMEND": true,
+        "CANCEL": true,
+        "copy_dict": true,
+        "get_number_format_info": true,
+        "strip_number_groups": true,
+        "print_table": true,
+        "Layout": true,
+        "web_form_settings": true,
+        "$c": true,
+        "$a": true,
+        "$i": true,
+        "$bg": true,
+        "$y": true,
+        "$c_obj": true,
+        "refresh_many": true,
+        "refresh_field": true,
+        "toggle_field": true,
+        "get_field_obj": true,
+        "get_query_params": true,
+        "unhide_field": true,
+        "hide_field": true,
+        "set_field_options": true,
+        "getCookie": true,
+        "getCookies": true,
+        "get_url_arg": true,
+        "md5": true,
+        "$": true,
+        "jQuery": true,
+        "moment": true,
+        "hljs": true,
+        "Awesomplete": true,
+        "Sortable": true,
+        "Showdown": true,
+        "Taggle": true,
+        "Gantt": true,
+        "Slick": true,
+        "Webcam": true,
+        "PhotoSwipe": true,
+        "PhotoSwipeUI_Default": true,
+        "io": true,
+        "JsBarcode": true,
+        "L": true,
+        "Chart": true,
+        "DataTable": true,
+        "Cypress": true,
+        "cy": true,
+        "it": true,
+        "describe": true,
+        "expect": true,
+        "context": true,
+        "before": true,
+        "beforeEach": true,
+        "after": true,
+        "qz": true,
+        "localforage": true,
+        "extend_cscript": true
+    }
+}

.flake8

@@ -0,0 +1,75 @@
+[flake8]
+ignore =
+    B001,
+    B007,
+    B009,
+    B010,
+    B950,
+    E101,
+    E111,
+    E114,
+    E116,
+    E117,
+    E121,
+    E122,
+    E123,
+    E124,
+    E125,
+    E126,
+    E127,
+    E128,
+    E131,
+    E201,
+    E202,
+    E203,
+    E211,
+    E221,
+    E222,
+    E223,
+    E224,
+    E225,
+    E226,
+    E228,
+    E231,
+    E241,
+    E242,
+    E251,
+    E261,
+    E262,
+    E265,
+    E266,
+    E271,
+    E272,
+    E273,
+    E274,
+    E301,
+    E302,
+    E303,
+    E305,
+    E306,
+    E402,
+    E501,
+    E502,
+    E701,
+    E702,
+    E703,
+    E741,
+    F401,
+    F403,
+    F405,
+    W191,
+    W291,
+    W292,
+    W293,
+    W391,
+    W503,
+    W504,
+    E711,
+    E129,
+    F841,
+    E713,
+    E712,
+    B028,
+
+max-line-length = 200
+exclude=,test_*.py

.pre-commit-config.yaml

@@ -0,0 +1,65 @@
+exclude: 'node_modules|.git'
+default_stages: [commit]
+fail_fast: false
+
+
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.3.0
+    hooks:
+      - id: trailing-whitespace
+        files: "frappe.*"
+        exclude: ".*json$|.*txt$|.*csv|.*md|.*svg"
+      - id: check-yaml
+      - id: no-commit-to-branch
+        args: ['--branch', 'develop']
+      - id: check-merge-conflict
+      - id: check-ast
+      - id: check-json
+      - id: check-toml
+      - id: check-yaml
+      - id: debug-statements
+
+  - repo: https://github.com/asottile/pyupgrade
+    rev: v2.34.0
+    hooks:
+      - id: pyupgrade
+        args: ['--py310-plus']
+
+  - repo: https://github.com/frappe/black
+    rev: 951ccf4d5bb0d692b457a5ebc4215d755618eb68
+    hooks:
+      - id: black
+
+  - repo: https://github.com/pre-commit/mirrors-prettier
+    rev: v2.7.1
+    hooks:
+      - id: prettier
+        types_or: [javascript]
+        # Ignore any files that might contain jinja / bundles
+        exclude: |
+            (?x)^(
+                frappe/public/dist/.*|
+                .*node_modules.*|
+                .*boilerplate.*|
+                frappe/www/website_script.js|
+                frappe/templates/includes/.*|
+                frappe/public/js/lib/.*
+            )$
+
+
+  - repo: https://github.com/PyCQA/isort
+    rev: 5.12.0
+    hooks:
+      - id: isort
+
+  - repo: https://github.com/PyCQA/flake8
+    rev: 5.0.4
+    hooks:
+      - id: flake8
+        additional_dependencies: ['flake8-bugbear',]
+
+ci:
+    autoupdate_schedule: weekly
+    skip: []
+    submodules: false

setup.py

@@ -1,4 +1,4 @@
-from setuptools import setup, find_packages
+from setuptools import find_packages, setup
 
 with open("requirements.txt") as f:
 	install_requires = f.read().strip().split("\n")
@@ -15,5 +15,5 @@
 	packages=find_packages(),
 	zip_safe=False,
 	include_package_data=True,
-	install_requires=install_requires
+	install_requires=install_requires,
 )

thai_tax/__init__.py

@@ -1,3 +1 @@
-
-__version__ = '0.0.1'
-
+__version__ = "0.0.1"

thai_tax/config/desktop.py

@@ -1,10 +1,5 @@
 from frappe import _
 
+
 def get_data():
-	return [
-		{
-			"module_name": "Thai Tax",
-			"type": "module",
-			"label": _("Thai Tax")
-		}
-	]
+	return [{"module_name": "Thai Tax", "type": "module", "label": _("Thai Tax")}]

thai_tax/config/docs.py

@@ -6,5 +6,6 @@
 # headline = "App that does everything"
 # sub_heading = "Yes, you got that right the first time, everything"
 
+
 def get_context(context):
 	context.brand_html = "Thai Tax"