🔀 Merge pull request #897 from marocchino/owner

✨ owner option

Author: marocchino

README.md

@@ -191,6 +191,10 @@ is explicitly passed, this problem can be solved by just removing it.
 
 **Optional** Pull request number for push event. Note that this has a **lower priority** than the number of a pull_request event.
 
+### `owner`
+
+**Optional** Another repository owner, If not set, the current repository owner is used by default. Note that when you trying changing a repo, be aware that `GITHUB_TOKEN` should also have permission for that repository.
+
 ### `repo`
 
 **Optional** Another repository name. Of limited use on GitHub enterprise. If not set, the current repository is used by default. Note that when you trying changing a repo, be aware that `GITHUB_TOKEN` should also have permission for that repository.

__tests__/config.test.ts

@@ -18,6 +18,7 @@ beforeEach(() => {
 afterEach(() => {
   jest.resetModules()
   delete process.env["GITHUB_REPOSITORY"]
+  delete process.env["INPUT_OWNER"]
   delete process.env["INPUT_REPO"]
   delete process.env["INPUT_HEADER"]
   delete process.env["INPUT_MESSAGE"]
@@ -38,10 +39,11 @@ afterEach(() => {
 })
 
 test("repo", async () => {
+  process.env["INPUT_OWNER"] = "jin"
   process.env["INPUT_REPO"] = "other"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
-    repo: {owner: "marocchino", repo: "other"},
+    repo: {owner: "jin", repo: "other"},
     header: "",
     append: false,
     recreate: false,

action.yml

@@ -59,8 +59,11 @@ inputs:
   number:
     description: "pull request number for push event"
     required: false
+  owner:
+    description: "Another repo owner, If not set, the current repo owner is used by default. Note that when you trying changing a repo, be aware that GITHUB_TOKEN should also have permission for that repository."
+    required: false
   repo:
-    description: "other repo name limited use on github enterprise. If not set, the current repo is used by default. Note that When you trying changing a repo, be aware that GITHUB_TOKEN should also use that repo's."
+    description: "Another repo name limited use on github enterprise. If not set, the current repo is used by default. Note that When you trying changing a repo, be aware that GITHUB_TOKEN should also use that repo's."
     required: false
   GITHUB_TOKEN:
     description: "The GitHub access token (e.g. secrets.GITHUB_TOKEN) used to create or update the comment. This defaults to {{ github.token }}."

dist/index.js

@@ -36,7 +36,7 @@ export const ignoreEmpty = core.getBooleanInput("ignore_empty", {
 
 function buildRepo(): {repo: string; owner: string} {
   return {
-    owner: context.repo.owner,
+    owner: core.getInput("owner", {required: false}) || context.repo.owner,
     repo: core.getInput("repo", {required: false}) || context.repo.repo
   }
 }