Fix: checking multiple files

[LoicRiegel]

Discussion #125

About the changes:

When users provide multiple files or patterns to check:

• duplicates are removed
• exit with status 1 if any pattern is an invalid pattern or does not match anything
• the files or patterns (valid or not) are returned in order (that's why I used IndexSet, hope it's okay to add it as a dependancy)

Results

You can test it yourself too, but here are a few screenshots:

User provides one valid file and a invalid glob pattern --> exit with 1

Check that duplicates are removed, and ``good2.py´` is analyzed first

Check that options, flags and runtime args are skipped - full example

Next steps

I'm waiting for your review, I'm new to Rust so feel free to tell me what could be improved / done in a more idiomatic way etc...

I ran cargo fmt and cargo clippy, but tell me if there are something else that I need to do.

And I will of course squash everything when we're good, and rebase if you want.

[LoicRiegel]

Thanks for your review. I did the fixes your suggested. The branch is rebased with main, and my fork is in sync. I can squash if you want, but you can also just "Squash & merge" from GitHub

[mtshiba]

OK, I'll do so.

[mtshiba]

Thank you!

[LoicRiegel]

You're welcome, happy to help!
The pre-commit hook should work now, I'll check that later

[LoicRiegel]

@mtshiba could you please make a new release of pylyzer? I need to have this fix on pypi for the pre-commit hook to work :)