filter author/committer with an array of string

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. */
@@ -125,8 +125,17 @@ function addOptional<Field extends string = DefaultField>(
   ] as const;
 
   for (let i = cmdOptional.length; i--; ) {
-    if (options[cmdOptional[i]]) {
-      commandWithOptions += ` --${cmdOptional[i]}="${options[cmdOptional[i]]}"`;
+    const flagOptions = options[cmdOptional[i]];
+    const commandOption = cmdOptional[i];
+
+    if (flagOptions && commandOption) {
+      if (Array.isArray(flagOptions)) {
+        flagOptions.map(flagOption => {
+          commandWithOptions += ` --${commandOption}="${flagOption}"`;
+        })
+      } else {
+        commandWithOptions += ` --${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");