Use glob for path param

This was done such that multiple files
can be loaded that match given glob
patterns. Existing behaviour is preserved
for exact match.

__tests__/assets/result2

@@ -0,0 +1 @@
+hey there

__tests__/config.test.ts

@@ -29,12 +29,11 @@ afterEach(() => {
   delete process.env["INPUT_PATH"]
 })
 
-test("repo", () => {
+test("repo", async () => {
   process.env["INPUT_REPO"] = "other"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "other"},
-    body: "",
     header: "",
     append: false,
     recreate: false,
@@ -45,13 +44,13 @@ test("repo", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("header", () => {
+test("header", async () => {
   process.env["INPUT_HEADER"] = "header"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "header",
     append: false,
     recreate: false,
@@ -62,13 +61,13 @@ test("header", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("append", () => {
+test("append", async () => {
   process.env["INPUT_APPEND"] = "true"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "",
     append: true,
     recreate: false,
@@ -79,13 +78,13 @@ test("append", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("recreate", () => {
+test("recreate", async () => {
   process.env["INPUT_RECREATE"] = "true"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "",
     append: false,
     recreate: true,
@@ -96,13 +95,13 @@ test("recreate", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("delete", () => {
+test("delete", async () => {
   process.env["INPUT_DELETE"] = "true"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "",
     append: false,
     recreate: false,
@@ -113,13 +112,13 @@ test("delete", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("hideOldComment", () => {
+test("hideOldComment", async () => {
   process.env["INPUT_HIDE"] = "true"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "",
     append: false,
     recreate: false,
@@ -130,13 +129,13 @@ test("hideOldComment", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("hideAndRecreate", () => {
+test("hideAndRecreate", async () => {
   process.env["INPUT_HIDE_AND_RECREATE"] = "true"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "",
     append: false,
     recreate: false,
@@ -147,13 +146,13 @@ test("hideAndRecreate", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("hideClassify", () => {
+test("hideClassify", async () => {
   process.env["INPUT_HIDE_CLASSIFY"] = "OFF_TOPIC"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "",
     append: false,
     recreate: false,
@@ -164,13 +163,13 @@ test("hideClassify", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
-test("hideDetails", () => {
+test("hideDetails", async () => {
   process.env["INPUT_HIDE_DETAILS"] = "true"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "",
     header: "",
     append: false,
     recreate: false,
@@ -181,14 +180,14 @@ test("hideDetails", () => {
     hideDetails: true,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("")
 })
 describe("path", () => {
-  test("when exists return content of a file", () => {
+  test("when exists return content of a file", async () => {
     process.env["INPUT_PATH"] = "./__tests__/assets/result"
     expect(require("../src/config")).toMatchObject({
       pullRequestNumber: expect.any(Number),
       repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-      body: "hi there\n",
       header: "",
       append: false,
       recreate: false,
@@ -199,14 +198,34 @@ describe("path", () => {
       hideDetails: false,
       githubToken: "some-token"
     })
+    expect(await require("../src/config").getBody()).toEqual("hi there\n")
   })
 
-  test("when not exists return null string", () => {
+  test("glob match files", async () => {
+    process.env["INPUT_PATH"] = "./__tests__/assets/*"
+    expect(require("../src/config")).toMatchObject({
+      pullRequestNumber: expect.any(Number),
+      repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
+      header: "",
+      append: false,
+      recreate: false,
+      deleteOldComment: false,
+      hideOldComment: false,
+      hideAndRecreate: false,
+      hideClassify: "OUTDATED",
+      hideDetails: false,
+      githubToken: "some-token"
+    })
+    expect(await require("../src/config").getBody()).toEqual(
+      "hi there\n\nhey there\n"
+    )
+  })
+
+  test("when not exists return null string", async () => {
     process.env["INPUT_PATH"] = "./__tests__/assets/not_exists"
     expect(require("../src/config")).toMatchObject({
       pullRequestNumber: expect.any(Number),
       repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-      body: "",
       header: "",
       append: false,
       recreate: false,
@@ -217,15 +236,15 @@ describe("path", () => {
       hideDetails: false,
       githubToken: "some-token"
     })
+    expect(await require("../src/config").getBody()).toEqual("")
   })
 })
 
-test("message", () => {
+test("message", async () => {
   process.env["INPUT_MESSAGE"] = "hello there"
   expect(require("../src/config")).toMatchObject({
     pullRequestNumber: expect.any(Number),
     repo: {owner: "marocchino", repo: "stick-pull-request-comment"},
-    body: "hello there",
     header: "",
     append: false,
     recreate: false,
@@ -236,4 +255,5 @@ test("message", () => {
     hideDetails: false,
     githubToken: "some-token"
   })
+  expect(await require("../src/config").getBody()).toEqual("hello there")
 })

action.yml

@@ -38,7 +38,10 @@ inputs:
     description: "comment message"
     required: false
   path:
-    description: "path to file containing comment message"
+    description: "glob path to file(s) containing comment message"
+    required: false
+  follow-symbolic-links:
+    description: "Indicates whether to follow symbolic links for path"
     required: false
   number:
     description: "pull request number for push event"

dist/licenses.txt

@@ -22,6 +22,18 @@ The above copyright notice and this permission notice shall be included in all c
 
 THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
+@actions/glob
+MIT
+The MIT License (MIT)
+
+Copyright 2019 GitHub
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
 @actions/http-client
 MIT
 Actions Http Client for Node.js
@@ -229,6 +241,31 @@ The above copyright notice and this permission notice shall be included in all c
 
 THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
+balanced-match
+MIT
+(MIT)
+
+Copyright (c) 2013 Julian Gruber <julian@juliangruber.com>
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the "Software"), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
+of the Software, and to permit persons to whom the Software is furnished to do
+so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+
+
 before-after-hook
 Apache-2.0
                                  Apache License
@@ -434,6 +471,53 @@ Apache-2.0
    limitations under the License.
 
 
+brace-expansion
+MIT
+MIT License
+
+Copyright (c) 2013 Julian Gruber <julian@juliangruber.com>
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+
+
+concat-map
+MIT
+This software is released under the MIT license:
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the "Software"), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
+the Software, and to permit persons to whom the Software is furnished to do so,
+subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
+FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
+IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+
 deprecation
 ISC
 The ISC License
@@ -478,6 +562,25 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 
+minimatch
+ISC
+The ISC License
+
+Copyright (c) Isaac Z. Schlueter and Contributors
+
+Permission to use, copy, modify, and/or distribute this software for any
+purpose with or without fee is hereby granted, provided that the above
+copyright notice and this permission notice appear in all copies.
+
+THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR
+IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+
+
 node-fetch
 MIT
 The MIT License (MIT)

package.json

@@ -28,6 +28,7 @@
   "dependencies": {
     "@actions/core": "^1.10.0",
     "@actions/github": "^5.1.1",
+    "@actions/glob": "^0.3.0",
     "@octokit/graphql-schema": "^12.11.0"
   },
   "devDependencies": {

src/config.ts

@@ -2,6 +2,7 @@ import * as core from "@actions/core"
 import {ReportedContentClassifiers} from "@octokit/graphql-schema"
 import {context} from "@actions/github"
 import {readFileSync} from "fs"
+import {create} from "@actions/glob"
 
 export const pullRequestNumber =
   context?.payload?.pull_request?.number ||
@@ -24,8 +25,6 @@ export const deleteOldComment = core.getBooleanInput("delete", {required: true})
 export const hideOldComment = core.getBooleanInput("hide", {required: true})
 export const githubToken = core.getInput("GITHUB_TOKEN", {required: true})
 
-export const body = buildBody()
-
 function buildRepo(): {repo: string; owner: string} {
   return {
     owner: context.repo.owner,
@@ -33,11 +32,19 @@ function buildRepo(): {repo: string; owner: string} {
   }
 }
 
-function buildBody(): string {
+export async function getBody(): Promise<string> {
-  const path = core.getInput("path", {required: false})
+  const pathInput = core.getMultilineInput("path", {required: false})
-  if (path) {
+  const followSymbolicLinks =
+    core.getInput("follow-symbolic-links").toLocaleUpperCase() !== "FALSE"
+  if (pathInput && pathInput.length > 0) {
     try {
-      return readFileSync(path, "utf-8")
+      const globber = await create(pathInput.join("\n"), {
+        followSymbolicLinks,
+        matchDirectories: false
+      })
+      return (await globber.glob())
+        .map(path => readFileSync(path, "utf-8"))
+        .join("\n")
     } catch (error) {
       if (error instanceof Error) {
         core.setFailed(error.message)

src/main.ts

@@ -2,7 +2,7 @@ import * as core from "@actions/core"
 import * as github from "@actions/github"
 import {
   append,
-  body,
+  getBody,
   deleteOldComment,
   githubToken,
   header,
@@ -30,6 +30,8 @@ async function run(): Promise<undefined> {
   }
 
   try {
+    const body = await getBody()
+
     if (!deleteOldComment && !hideOldComment && !body) {
       throw new Error("Either message or path input is required")
     }

yarn.lock

@@ -2,7 +2,7 @@
 # yarn lockfile v1
 
 
-"@actions/core@^1.10.0":
+"@actions/core@^1.10.0", "@actions/core@^1.2.6":
   version "1.10.0"
   resolved "https://registry.yarnpkg.com/@actions/core/-/core-1.10.0.tgz#44551c3c71163949a2f06e94d9ca2157a0cfac4f"
   integrity sha512-2aZDDa3zrrZbP5ZYg159sNoLRb61nQ7awl5pSvIq5Qpj81vwDzdMRKzkWJGJuwVvWpvZKx7vspJALyvaaIQyug==
@@ -20,6 +20,14 @@
     "@octokit/plugin-paginate-rest" "^2.17.0"
     "@octokit/plugin-rest-endpoint-methods" "^5.13.0"
 
+"@actions/glob@^0.3.0":
+  version "0.3.0"
+  resolved "https://registry.yarnpkg.com/@actions/glob/-/glob-0.3.0.tgz#beb5282a8cd936baadbe99fbb1ccd5eaf2d1fc28"
+  integrity sha512-tJP1ZhF87fd6LBnaXWlahkyvdgvsLl7WnreW1EZaC8JWjpMXmzqWzQVe/IEYslrkT9ymibVrKyJN4UMD7uQM2w==
+  dependencies:
+    "@actions/core" "^1.2.6"
+    minimatch "^3.0.4"
+
 "@actions/http-client@^2.0.1":
   version "2.0.1"
   resolved "https://registry.yarnpkg.com/@actions/http-client/-/http-client-2.0.1.tgz#873f4ca98fe32f6839462a6f046332677322f99c"