refactor: single AXL module (#654)

users shouldn't have to dep on rules_lint twice

Author: alexeagle

.github/workflows/ci.yaml

@@ -65,7 +65,8 @@ jobs:
         run: aspect lint
       - name: Format
         working-directory: example
-        run: aspect format
+        # Only format one language to keep CI time reasonable
+        run: aspect format --format_target=//tools/format:format_Shell_with_shfmt --all_files=true
   test:
     strategy:
       matrix:

MODULE.aspect

@@ -0,0 +1,2 @@
+use_task("lint/lint.axl", "lint")
+use_task("format/format.axl", "format")

example/MODULE.aspect

@@ -1,11 +1,5 @@
 axl_local_dep(
-    name = "lint",
-    path = "../lint",
-    auto_use_tasks = True,
-)
-
-axl_local_dep(
-    name = "format",
-    path = "../format",
+    name = "rules_lint",
+    path = "..",
     auto_use_tasks = True,
 )

format/MODULE.aspect

@@ -84,13 +84,16 @@ def impl(ctx: TaskContext) -> int:
 
     formatter = build_formatter(ctx)
 
-    changed_files = find_changed_files(out, ctx.std.process)
-    out.write("Formatting changed files:\n%s" % changed_files)
+    format_args = []
+    if not ctx.args.all_files:
+        changed_files = find_changed_files(out, ctx.std.process)
+        out.write("Formatting changed files:\n%s" % changed_files)
+        format_args = changed_files.split("\n")
 
     exit = bazel_run(
         ctx,
         entrypoint = formatter,
-        args = changed_files.split("\n"),
+        args = format_args,
     ).wait()
 
     if not exit.success:
@@ -101,6 +104,7 @@ def impl(ctx: TaskContext) -> int:
 format = task(
     implementation = impl,
     args = {
+        "all_files": args.boolean(default = False),
         "format_target": args.string(default = "//tools/format"),
     },
 )