Separate integration workflow

jclem · jclem · commit d73e75dea885 · 2020-02-27T17:52:43.000-05:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -3,20 +3,7 @@ on:
   pull_request: {branches: master}
 
 jobs:
-  integration:
-    runs-on: ubuntu-latest
-    steps:
-      - name: output-set
-        uses: actions/github-script@master
-        with:
-          script: |
-            core.setOutput('result', 'test')
-      - run: |
-          if [[ "${{steps.output-set.outputs.result}}" != "test" ]]; then
-            exit 1
-          fi
-
-  jest:
+  ci:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
diff --git a/.github/workflows/integration.yml b/.github/workflows/integration.yml
@@ -0,0 +1,16 @@
+on:
+  push: {branches: integration-workflow}
+
+jobs:
+  integration:
+    runs-on: ubuntu-latest
+    steps:
+      - id: output-set
+        uses: actions/github-script@integration-workflow
+        with:
+          script: return 'test'
+          result-encoding: string
+      - run: |
+          if [[ "${{steps.output-set.outputs.result}}" != "test" ]]; then
+            exit 1
+          fi
diff --git a/dist/index.js b/dist/index.js
@@ -9178,7 +9178,7 @@ async function main() {
     const github = new lib_github.GitHub(token, opts);
     const script = Object(core.getInput)('script', { required: true });
     // Using property/value shorthand on `require` (e.g. `{require}`) causes compilatin errors.
-    const result = callAsyncFunction({ require: __webpack_require__(875), github: github, context: lib_github.context, core: core }, script);
+    const result = await callAsyncFunction({ require: __webpack_require__(875), github: github, context: lib_github.context, core: core }, script);
     let encoding = Object(core.getInput)('result-encoding');
     encoding = encoding ? encoding : 'json';
     let output;
diff --git a/src/main.ts b/src/main.ts
@@ -18,7 +18,7 @@ async function main() {
   const script = core.getInput('script', {required: true})
 
   // Using property/value shorthand on `require` (e.g. `{require}`) causes compilatin errors.
-  const result = callAsyncFunction(
+  const result = await callAsyncFunction(
     {require: require, github: github, context: context, core: core},
     script
   )

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ async function main() {`
`18`	`18`	`const script = core.getInput('script', {required: true})`
`19`	`19`
`20`	`20`	// Using property/value shorthand on `require` (e.g. `{require}`) causes compilatin errors.
`21`		`- const result = callAsyncFunction(`
	`21`	`+ const result = await callAsyncFunction(`
`22`	`22`	`{require: require, github: github, context: context, core: core},`
`23`	`23`	`script`
`24`	`24`	`)`