filter author/committer with an array of strings

README.md

@@ -101,6 +101,7 @@ Show commits older than a specific date.
 ### author/committer
 
 Limit the commits output to ones with author/committer header lines that match the specified pattern.
+Can also be an array of patterns/strings.
 
 ### nameStatus
 

src/index.ts

@@ -85,9 +85,9 @@ export interface GitlogOptions<Fields extends string = DefaultField> {
   /** File filter for the git log command */
   file?: string;
   /** Limit the commits output to ones with author header lines that match the specified pattern. */
-  author?: string;
+  author?: string | string[];
   /** Limit the commits output to ones with committer header lines that match the specified pattern. */
-  committer?: string;
+  committer?: string | string[];
   /** Show commits more recent than a specific date. */
   since?: string;
   /** Show commits more recent than a specific date. */
@@ -124,9 +124,21 @@ function addOptional<Field extends string = DefaultField>(
     "committer",
   ] as const;
 
+  const addOption = (option: string) => (value: string) => {
+    commandWithOptions += ` --${option}="${value}"`;
+  };
+
   for (let i = cmdOptional.length; i--; ) {
-    if (options[cmdOptional[i]]) {
-      commandWithOptions += ` --${cmdOptional[i]}="${options[cmdOptional[i]]}"`;
+    const commandOption = cmdOptional[i];
+    const flagOptions = options[commandOption];
+
+    if (flagOptions) {
+      if (Array.isArray(flagOptions)) {
+        const addMultiOption = addOption(commandOption);
+        flagOptions.forEach(addMultiOption);
+      } else {
+        addOption(commandOption)(flagOptions);
+      }
     }
   }
 

test/gitlog.test.ts

@@ -210,25 +210,51 @@ describe("gitlog", () => {
     });
   });
 
+  it("returns commits only by multiple authors", (done) => {
+    const command =
+      `cd ${testRepoLocation}` +
+      "&& touch new-file" +
+      "&& git add new-file" +
+      '&& git commit -m "New commit" --author="A U Thor <[email protected]>"' +
+      "&& touch loki-file" +
+      "&& git add loki-file" +
+      '&& git commit -m "loki commit" --author="A U Loki <[email protected]>"';
+    const authors = ["A U Thor", "A U Loki"];
+
+    // Adding a new commit by different author
+    exec(command, () => {
+      const commits = gitlog({
+        repo: testRepoLocation,
+        author: authors,
+        fields: ["authorName"],
+      });
+
+      expect.assertions(2);
+      commits.forEach((commit) => {
+        expect(authors.includes(commit.authorName)).toBe(true);
+      });
+
+      done();
+    });
+  });
+
   it("returns commits only by committer", (done) => {
-    const defaults = ["committerName"] as const;
     const command =
       `cd ${testRepoLocation} ` +
-      `&& touch new-file ` +
-      `&& git add new-file ` +
-      `&& git commit -m "New commit" ` +
-      `--committer="A U Thor <[email protected]>"`;
-    const committer = "Your Name";
+      "&& touch new-file " +
+      "&& git add new-file " +
+      '&& git -c "user.name=A U Thor" -c "[email protected]" commit -m "New commit"';
+    const committer = "A U Thor";
 
     // Adding a new commit by different author
     exec(command, () => {
       const commits = gitlog({
         repo: testRepoLocation,
         committer,
-        fields: defaults,
+        fields: ["committerName"],
       });
 
-      expect.assertions(10);
+      expect.assertions(1);
       commits.forEach((commit) => {
         expect(commit.committerName).toBe(committer);
       });
@@ -237,6 +263,34 @@ describe("gitlog", () => {
     });
   });
 
+  it("returns commits only by multiple committers", (done) => {
+    const command =
+      `cd ${testRepoLocation} ` +
+      "&& touch new-file " +
+      "&& git add new-file " +
+      '&& git -c "user.name=A U Thor" -c "[email protected]"  commit -m "New commit"' +
+      "&& touch loki-file " +
+      "&& git add loki-file " +
+      '&& git -c "user.name=A U Loki" -c "[email protected]"  commit -m "loki commit"';
+    const committer = ["A U Thor", "A U Loki"];
+
+    // Adding a new commit by different author
+    exec(command, () => {
+      const commits = gitlog({
+        repo: testRepoLocation,
+        committer,
+        fields: ["committerName", "subject"],
+      });
+
+      expect.assertions(2);
+      commits.forEach((commit) => {
+        expect(committer.includes(commit.committerName)).toBe(true);
+      });
+
+      done();
+    });
+  });
+
   it("returns A status for files that are added", () => {
     const commits = gitlog({ repo: testRepoLocation });
     expect(commits[1].status[0]).toBe("A");