chore: handle CARGO_TARGET_DIR in tools/linter (#1179)

.github/workflows/ci.yml

@@ -63,7 +63,7 @@ jobs:
 
  - name: Lint
  if: contains(matrix.os, 'ubuntu')
- run: deno run --allow-run ./tools/lint.ts --release
+ run: deno run --allow-run --allow-env ./tools/lint.ts --release
 
  - name: Check if www/static/docs.json is up-to-date
  if: contains(matrix.os, 'ubuntu')

README.md

@@ -196,7 +196,7 @@ Before submitting, please make sure the following is done:
 2. There are tests that cover the changes.
 3. Ensure `cargo test` passes.
 4. Format your code with `deno run --allow-run tools/format.ts`
-5. Make sure `deno run --allow-run tools/lint.ts` passes.
+5. Make sure `deno run --allow-run --allow-env tools/lint.ts` passes.
 6. If you've added a new rule:
  1. Run `cargo build --example dlint --all-features`
  2. Update docs by running the generated binary with these arguments

tools/lint.ts

@@ -28,7 +28,10 @@ if (o1.code !== 0) {
 
 console.log("deno lint");
 
-const dlint = `./target/${release ? "release" : "debug"}/examples/dlint`;
+const cargoTargetDir = Deno.env.get("CARGO_TARGET_DIR") || "./target";
+const dlint = `${cargoTargetDir}/${
+ release ? "release" : "debug"
+}/examples/dlint`;
 const p2 = new Deno.Command(dlint, {
  args: ["run", "benchmarks/benchmarks.ts"],
  stdin: "null",