Merge pull request #1005 from crazy-max/util-inputlist-comment

util: add commentNoInfix support to getInputList

Author: crazy-max

__tests__/util.test.ts

@@ -87,6 +87,12 @@ describe('getInputList', () => {
     expect(res).toEqual(['foo=bar', 'bar=qux']);
   });
 
+  it('multiline with comment and no infix comment', async () => {
+    setInput('labels', '# ignore this\nfoo=bar\nbar=qux#baz');
+    const res = Util.getInputList('labels', {comment: '#', commentNoInfix: true});
+    expect(res).toEqual(['foo=bar', 'bar=qux#baz']);
+  });
+
   it('different new lines and ignoring comma correctly', async () => {
     setInput('cache-from', 'user/app:cache\r\ntype=local,src=path/to/dir');
     const res = Util.getInputList('cache-from', {ignoreComma: true});

src/util.ts

@@ -24,6 +24,7 @@ import {parse} from 'csv-parse/sync';
 export interface ListOpts {
   ignoreComma?: boolean;
   comment?: string;
+  commentNoInfix?: boolean;
   quote?: string | boolean | Buffer | null;
 }
 
@@ -42,6 +43,7 @@ export class Util {
       columns: false,
       relaxQuotes: true,
       comment: opts?.comment,
+      comment_no_infix: opts?.commentNoInfix,
       relaxColumnCount: true,
       skipEmptyLines: true,
       quote: opts?.quote