Don't add installation to path

Unlike the `arduino/setup-arduino-cli` action this was based on, this is a standalone action. So adding the installation
to the system path would only serve to pollute the runner environment.

Author: per1234

__tests__/main.test.ts

@@ -43,7 +43,7 @@ describe("installer tests", () => {
   });
 
   it("Downloads version of arduino-lint if no matching version is installed", async () => {
-    await installer.getArduinoLint("0.4.0");
+    const toolPath = await installer.getArduinoLint("0.4.0");
     const bindir = path.join(
       toolDir,
       "arduino-lint",
@@ -52,6 +52,7 @@ describe("installer tests", () => {
       "arduino-lint"
     );
 
+    expect(toolPath == path.join(bindir, "arduino-lint")).toBe(true);
     expect(fs.existsSync(`${bindir}.complete`)).toBe(true);
 
     if (IS_WINDOWS) {

dist/index.js

@@ -80,7 +80,7 @@ function getArduinoLint(version) {
             toolPath = yield downloadRelease(version);
             core.debug("arduino-lint cached under " + toolPath);
         }
-        core.addPath(`${toolPath}/arduino-lint`);
+        return path.join(toolPath, "arduino-lint");
     });
 }
 exports.getArduinoLint = getArduinoLint;
@@ -272,8 +272,9 @@ function run() {
             const reportFile = core.getInput("report-file");
             const verbose = core.getInput("verbose");
             const official = core.getInput("official");
+            let toolPath = "arduino-lint";
             if (version) {
-                yield installer.getArduinoLint(version);
+                toolPath = yield installer.getArduinoLint(version);
             }
             const execArgs = [
                 "--compliance",
@@ -303,7 +304,7 @@ function run() {
                     ARDUINO_LINT_OFFICIAL: official,
                 },
             };
-            yield exec.exec("arduino-lint", execArgs, options);
+            yield exec.exec(toolPath, execArgs, options);
         }
         catch (error) {
             core.setFailed(error.message);

src/installer.ts

@@ -33,7 +33,7 @@ interface ITaskRef {
   ref: string;
 }
 
-export async function getArduinoLint(version: string) {
+export async function getArduinoLint(version: string): Promise<string> {
   // resolve the version number
   const targetVersion = await computeVersion(version);
   if (targetVersion) {
@@ -50,7 +50,7 @@ export async function getArduinoLint(version: string) {
     core.debug("arduino-lint cached under " + toolPath);
   }
 
-  core.addPath(`${toolPath}/arduino-lint`);
+  return path.join(toolPath, "arduino-lint");
 }
 
 async function downloadRelease(version: string): Promise<string> {

src/main.ts

@@ -14,8 +14,9 @@ async function run() {
     const verbose = core.getInput("verbose");
     const official = core.getInput("official");
 
+    let toolPath = "arduino-lint";
     if (version) {
-      await installer.getArduinoLint(version);
+      toolPath = await installer.getArduinoLint(version);
     }
 
     const execArgs = [
@@ -53,7 +54,7 @@ async function run() {
       },
     };
 
-    await exec.exec("arduino-lint", execArgs, options);
+    await exec.exec(toolPath, execArgs, options);
   } catch (error) {
     core.setFailed(error.message);
   }