Textfile collector: collect files from multiple paths

[DiniFarb]

Regarding #1236 and more

Added Features:

• collect files from multiple paths
• walkDir func for all paths to collect files from subdirs
• new flag command collector.textfile.directories

About the collector.textfile.directories commmand:
For now it is possible to use collector.textfile.directories, collector.textfile.directory or both. They have the same functionality and in case both commands would be used, the dirs are concatenated. I am not sure if that is ok so lets discuss 😉

As soon all is settled, I'll update the docs - collector.textfile.md 😊

[breed808]

Thanks for submitting this!

Is the plan to eventually deprecate the collector.textfile.directory flag in favour of the collector.textfile.directories flag? If so, it'd be worth adding a deprecation warning to the old flag (see the process collector for an example).

[DiniFarb]

I wasn't quite sure about deprecating the collector.textfile.directory flag. Technically both flags could stay but it could lead to confusion. Maybe @JDA88 or @LaurenceChau want to give also their input about what flag would be best?

I think the best way to go is as mentioned from @breed808 - adding a deprecation warning to collector.textfile.directory flag and then remove it a few versions later.

[JDA88]

I think the best way to go is as mentioned from @breed808 - adding a deprecation warning to collector.textfile.directory flag and then remove it a few versions later.

I agree, as long as the current --collector.textfile.directory continue to work like today for at least a few versions

[breed808]

Looking good! I have one minor concern for the deprecated flag, but everything else is fine.
Thanks for updating this.

EDIT: It'd also be worth rebasing the feature branch on master to remove the merge commits.

[DiniFarb]

I think this is ready to merge or does it need some further checks/reviews ?

[breed808]

Looks good! Thanks for your efforts with this one.