fix: add typing for outputs

Expensify · Feb 14, 2024 · 738defb · 738defb
1 parent 32ed6d6
commit 738defb
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 3 deletions.
diff --git a/.github/actions/javascript/getGraphiteString/action.yml b/.github/actions/javascript/getGraphiteString/action.yml
@@ -1,5 +1,8 @@
 name: 'Get and Save graphite string'
 description: 'Parse reassure output.json file and create string which can be sent to the graphite server'
+outputs:
+  GRAPHITE_STRING:
+    description: String with reassure data which can be directly sent to the graphite server
 runs:
   using: 'node20'
   main: './index.js'
diff --git a/.github/actions/javascript/getGraphiteString/getGraphiteString.js b/.github/actions/javascript/getGraphiteString/getGraphiteString.js
@@ -35,7 +35,7 @@ const run = () => {
         .join('\n');
 
     // Set generated graphite string to the github variable
-    core.setOutput('graphiteString', graphiteString);
+    core.setOutput('GRAPHITE_STRING', graphiteString);
 };
 
 if (require.main === module) {

diff --git a/.github/actions/javascript/getGraphiteString/index.js b/.github/actions/javascript/getGraphiteString/index.js
@@ -44,7 +44,7 @@ const run = () => {
         .join('\n');
 
     // Set generated graphite string to the github variable
-    core.setOutput('graphiteString', graphiteString);
+    core.setOutput('GRAPHITE_STRING', graphiteString);
 };
 
 if (require.main === require.cache[eval('__filename')]) {

diff --git a/.github/workflows/reassurePerformanceTests.yml b/.github/workflows/reassurePerformanceTests.yml
@@ -53,4 +53,4 @@ jobs:
       - name: Send graphite data
         # run only when merged to master
         if: github.event.pull_request.merged == true && github.event.pull_request.base.ref == 'main'
-        run: echo -e "${{ steps.saveGraphiteString.outputs.graphiteString }}" | nc "$GRAFHITE_SERVER" "$GRAFHITE_PORT"
+        run: echo -e "${{ steps.saveGraphiteString.outputs.GRAPHITE_STRING }}" | nc "$GRAFHITE_SERVER" "$GRAFHITE_PORT"