feat: Allow multiple paths with --filter flag

[rsinnet]

Pull Request

Related Github Issues

• [none]

Description

Update the type of the --filter flag so multiple paths or glob expressions can be passed as a comma-separated list.

Security Implications

• [none]

System Availability

• [none]

[rsinnet]

Do I need to do anything to get this assigned?

[amontalban]

I really need this feature 🙏

@dmattia what is needed to have this merged? Thank you!

[Almenon]

@amontalban I reviewed the code and it looks good. I still need to pass it by chatgpt. The code fails a few tests when I run it locally on Windows. It's been a long time since this PR was created, so the PR will need to be rebased on top of master and any test failures fixed. I would wait a bit* to see if @rsinnet will take this on, otherwise you can create your own PR.

* wait time depending on how patient you are, I would suggest a day at least

[rsinnet]

@amontalban I reviewed the code and it looks good. I still need to pass it by chatgpt. The code fails a few tests when I run it locally on Windows. It's been a long time since this PR was created, so the PR will need to be rebased on top of master and any test failures fixed. I would wait a bit* to see if @rsinnet will take this on, otherwise you can create your own PR.

• wait time depending on how patient you are, I would suggest a day at least

Sorry, guys. I left the organization and no longer have push access to the branch. Maybe @shaurya-pednekar or @imjaya can help out here?

[Almenon]

The code fails a few tests when I run it locally on Windows

Ignore this part, my bad. This was because #311 was not merged in yet. Once I merged it in all the tests passed with a copy-pasted set of your changes 🎉 . The PR will still need to be rebased or recreated to make sure the tests pass in CI.

[amontalban]

The code fails a few tests when I run it locally on Windows

Ignore this part, my bad. This was because #311 was not merged in yet. Once I merged it in all the tests passed with a copy-pasted set of your changes 🎉 . The PR will still need to be rebased or recreated to make sure the tests pass in CI.

@Almenon created #328 with latest master code.

Thank you for checking this out!

[Almenon]

I just merged in the new PR, closing. Thanks for the contribution!