Merge pull request #31 from crazy-max/fix-setoutput

Fix setOutput behavior

.github/workflows/ci.yml

@@ -43,3 +43,31 @@ jobs:
       -
         name: Dump context
         uses: crazy-max/ghaction-dump-context@v1
+
+  error:
+    runs-on: ubuntu-latest
+    steps:
+      -
+        name: Checkout
+        uses: actions/checkout@v2
+      -
+        name: Stop docker
+        run: |
+          sudo systemctl stop docker
+      -
+        name: Set up QEMU
+        id: qemu
+        continue-on-error: true
+        uses: ./
+      -
+        name: Check
+        run: |
+          echo "${{ toJson(steps.qemu) }}"
+          if [ "${{ steps.qemu.outcome }}" != "failure" ] || [ "${{ steps.qemu.conclusion }}" != "success" ]; then
+            echo "::error::Should have failed"
+            exit 1
+          fi
+      -
+        name: Dump context
+        if: always()
+        uses: crazy-max/ghaction-dump-context@v1

README.md

@@ -19,7 +19,6 @@ ___
 * [Customizing](#customizing)
   * [inputs](#inputs)
 * [Keep up-to-date with GitHub Dependabot](#keep-up-to-date-with-github-dependabot)
-* [Limitation](#limitation)
 
 ## Usage
 
@@ -82,7 +81,3 @@ updates:
     schedule:
       interval: "daily"
 ```
-
-## Limitation
-
-This action is only available for Linux [virtual environments](https://docs.github.com/en/actions/reference/virtual-environments-for-github-hosted-runners#supported-virtual-environments-and-hardware-resources).

dist/index.js

@@ -90,17 +90,17 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-const os = __importStar(__webpack_require__(87));
 const mexec = __importStar(__webpack_require__(757));
 const core = __importStar(__webpack_require__(186));
 const exec = __importStar(__webpack_require__(514));
+const command_1 = __webpack_require__(241);
 function run() {
     return __awaiter(this, void 0, void 0, function* () {
         try {
-            if (os.platform() !== 'linux') {
+            core.startGroup(`Docker info`);
-                core.setFailed('Only supported on linux platform');
+            yield exec.exec('docker', ['version']);
-                return;
+            yield exec.exec('docker', ['info']);
-            }
+            core.endGroup();
             const image = core.getInput('image') || 'tonistiigi/binfmt:latest';
             const platforms = core.getInput('platforms') || 'all';
             core.startGroup(`Pulling binfmt Docker image`);
@@ -116,7 +116,7 @@ function run() {
                 }
                 const platforms = JSON.parse(res.stdout.trim());
                 core.info(`${platforms.supported.join(',')}`);
-                core.setOutput('platforms', platforms.supported.join(','));
+                setOutput('platforms', platforms.supported.join(','));
             });
             core.endGroup();
         }
@@ -125,6 +125,10 @@ function run() {
         }
     });
 }
+// FIXME: Temp fix https://github.com/actions/toolkit/issues/777
+function setOutput(name, value) {
+    command_1.issueCommand('set-output', { name }, value);
+}
 run();
 //# sourceMappingURL=main.js.map
 
@@ -854,6 +858,7 @@ exports.getInput = getInput;
  */
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 function setOutput(name, value) {
+    process.stdout.write(os.EOL);
     command_1.issueCommand('set-output', { name }, value);
 }
 exports.setOutput = setOutput;

package.json

@@ -26,7 +26,7 @@
   ],
   "license": "Apache-2.0",
   "dependencies": {
-    "@actions/core": "^1.2.6",
+    "@actions/core": "^1.2.7",
     "@actions/exec": "^1.0.4"
   },
   "devDependencies": {

src/main.ts

@@ -1,7 +1,7 @@
-import * as os from 'os';
 import * as mexec from './exec';
 import * as core from '@actions/core';
 import * as exec from '@actions/exec';
+import {issueCommand} from '@actions/core/lib/command';
 
 interface Platforms {
   supported: string[];
@@ -10,10 +10,10 @@ interface Platforms {
 
 async function run(): Promise<void> {
   try {
-    if (os.platform() !== 'linux') {
+    core.startGroup(`Docker info`);
-      core.setFailed('Only supported on linux platform');
+    await exec.exec('docker', ['version']);
-      return;
+    await exec.exec('docker', ['info']);
-    }
+    core.endGroup();
 
     const image: string = core.getInput('image') || 'tonistiigi/binfmt:latest';
     const platforms: string = core.getInput('platforms') || 'all';
@@ -33,7 +33,7 @@ async function run(): Promise<void> {
       }
       const platforms: Platforms = JSON.parse(res.stdout.trim());
       core.info(`${platforms.supported.join(',')}`);
-      core.setOutput('platforms', platforms.supported.join(','));
+      setOutput('platforms', platforms.supported.join(','));
     });
     core.endGroup();
   } catch (error) {
@@ -41,4 +41,9 @@ async function run(): Promise<void> {
   }
 }
 
+// FIXME: Temp fix https://github.com/actions/toolkit/issues/777
+function setOutput(name: string, value: any): void {
+  issueCommand('set-output', {name}, value);
+}
+
 run();

yarn.lock

@@ -2,10 +2,10 @@
 # yarn lockfile v1
 
 
-"@actions/core@^1.2.6":
+"@actions/core@^1.2.7":
-  version "1.2.6"
+  version "1.2.7"
-  resolved "https://registry.yarnpkg.com/@actions/core/-/core-1.2.6.tgz#a78d49f41a4def18e88ce47c2cac615d5694bf09"
+  resolved "https://registry.yarnpkg.com/@actions/core/-/core-1.2.7.tgz#594f8c45b213f0146e4be7eda8ae5cf4e198e5ab"
-  integrity sha512-ZQYitnqiyBc3D+k7LsgSBmMDVkOVidaagDG7j3fOym77jNunWRuYx7VSHa9GNfFZh+zh61xsCjRj4JxMZlDqTA==
+  integrity sha512-kzLFD5BgEvq6ubcxdgPbRKGD2Qrgya/5j+wh4LZzqT915I0V3rED+MvjH6NXghbvk1MXknpNNQ3uKjXSEN00Ig==
 
 "@actions/exec@^1.0.4":
   version "1.0.4"