Fix separator naming

Fix wrong default for pullID
Gamer025 · Oct 16, 2021 · 1ff559d · 1ff559d
1 parent e75696a
commit 1ff559d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/CodeOwnersParser/ActionInputs.cs b/CodeOwnersParser/ActionInputs.cs
@@ -46,7 +46,7 @@ public string Name
            Required = false,
            HelpText = "String used to seperate multiple owners in the output.",
            Default = " ")]
-        public string seperator { get; set; } = null!;
+        public string separator { get; set; } = null!;
 
         [Option('p', "prefix",
            Required = false,

diff --git a/CodeOwnersParser/Program.cs b/CodeOwnersParser/Program.cs
@@ -40,7 +40,7 @@ static void NotifyOwners(ActionInputs inputs)
         ownersWithModifiedFiles = ownersWithModifiedFiles.Except(notifiedOwners).ToList();
     }
 
-    string output = String.Join(inputs.seperator, ownersWithModifiedFiles);
+    string output = String.Join(inputs.separator, ownersWithModifiedFiles);
 
     Console.WriteLine($"Owners with file changes: {output}");
     Console.WriteLine($"::set-output name=owners::{output}");

diff --git a/action.yml b/action.yml
@@ -26,9 +26,9 @@ inputs:
     default: "/.github/CODEOWNERS"
   pullID:
     description:
-      "ID of the PR to process. Assign from `github.event_path.pull_request.number`."
+      "ID of the PR to process. Assign from `github.event.pull_request.number`."
     required: true
-    default: ${{ github.event_path.pull_request.number }}
+    default: ${{ github.event.pull_request.number }}
   separator:
     description:
       "Separator used if multiple owners are returned. Defaults to space."