Merge pull request #8 from youknowone/python-lint

Add python lint

Author: youknowone

.github/workflows/ci.yaml

@@ -54,3 +54,11 @@ jobs:
         run: cargo fmt --all -- --check
       - name: run clippy
         run: cargo clippy --all --all-features -- -Dwarnings
+
+      - uses: actions/setup-python@v4
+        with:
+          python-version: "3.11"
+      - name: install ruff
+        run: python -m pip install ruff
+      - name: run python lint
+        run: ruff --ignore=E501 ast --show-source

ast/asdl.py

@@ -176,7 +176,7 @@ def visitConstructor(self, cons, name):
 
     def visitField(self, field, name):
         key = str(field.type)
-        l = self.types.setdefault(key, [])
+        l = self.types.setdefault(key, [])  # noqa
         l.append(name)
 
     def visitProduct(self, prod, name):
@@ -193,7 +193,7 @@ def check(mod):
     v.visit(mod)
 
     for t in v.types:
-        if t not in mod.types and not t in builtin_types:
+        if t not in mod.types and t not in builtin_types:
             v.errors += 1
             uses = ", ".join(v.types[t])
             print('Undefined type {}, used in {}'.format(t, uses))

ast/asdl_rs.py

@@ -447,7 +447,7 @@ def gen_classdef(self, name, fields, attrs, depth, base="AstNode"):
         self.emit(f"struct {structname};", depth)
         self.emit("#[pyclass(flags(HAS_DICT, BASETYPE))]", depth)
         self.emit(f"impl {structname} {{", depth)
-        self.emit(f"#[extend_class]", depth + 1)
+        self.emit("#[extend_class]", depth + 1)
         self.emit(
             "fn extend_class_with_fields(ctx: &Context, class: &'static Py<PyType>) {",
             depth + 1,