fs should be optional in walkSync.Options

[kturney]

Ran into the walkSync.Options taken by this.input.entries in a broccoli plugin now saying that fs is required if an options object is provided.

example of what stopped working:

const sources = this.input
      .entries('.', {
        directories: false,
        globs: this.themeInputs.map((dir) => `${dir}/*.json`),
      })
      .map((entry) => entry.fullPath);

now gets error:

error TS2345: Argument of type '{ directories: false; globs: string[]; }' is not assignable to parameter of type 'Options'.
  Property 'fs' is missing in type '{ directories: false; globs: string[]; }' but required in type 'Options'.

33       .entries('.', {
                       ~
34         directories: false,
   ~~~~~~~~~~~~~~~~~~~~~~~~~~~
35         globs: this.themeInputs.map((dir) => `${dir}/*.json`),
   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
36       })
   ~~~~~~~

[kturney]

Any chance of getting this merged and published?

[kturney]

Hi all, any updates I can provide here?