add better error for pullRequestId being undefined (#5)

* add better error for pullRequestId being undefined

* version bump

* PR comment

Author: Germandrummer92

automagicallyexecute/.eslintrc.json

@@ -11,6 +11,7 @@
   },
   "ignorePatterns": ["node_modules"],
   "rules": {
+    "i18n-text/no-en": "off",
     "@typescript-eslint/no-floating-promises": "warn",
     "prettier/prettier": "warn",
     "no-console": "warn"

automagicallyexecute/index.ts

@@ -40,6 +40,15 @@ const run = async (): Promise<void> => {
       pullRequestId: getVariable('System.PullRequest.PullRequestId')
     }
 
+    if (!context.pullRequestId) {
+      setResult(
+        TaskResult.Failed,
+        'System.PullRequest.PullRequestId variable not available. ' +
+          'Make sure you run this task in a PR build validation pipeline, ' +
+          'otherwise we cannot comment back the test results'
+      )
+    }
+
     debug(JSON.stringify({executeUrl, context}, null, 2))
 
     // https://github.com/microsoft/azure-pipelines-task-lib/issues/579

automagicallyexecute/task.json

@@ -9,7 +9,7 @@
   "version": {
     "Major": 1,
     "Minor": 1,
-    "Patch": 1
+    "Patch": 2
   },
   "instanceNameFormat": "Execute Automagically",
   "inputs": [

vss-extension.json

@@ -2,7 +2,7 @@
   "manifestVersion": 1,
   "id": "automagically-execute-task",
   "name": "octomind automagically execute",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "publisher": "octomind",
   "targets": [
     {