statix icon indicating copy to clipboard operation
statix copied to clipboard
verified publisher icon nerdypepper

feat: allow user to pass multiple files to `fix` and `check` commands

Open sinrohit-desco opened this issue 8 months ago • 6 comments

Resolves https://github.com/oppiliappan/statix/issues/118

sinrohit-desco avatar Aug 13 '25 07:08 sinrohit-desco

I think we should have this. How about a test though?

mightyiam avatar Aug 17 '25 15:08 mightyiam

I think we should have this. How about a test though?

Yeah, definitely but before that I want to rework the changes as the program needs to respect each .gitignore files.

sinrohit-desco avatar Aug 18 '25 06:08 sinrohit-desco

I completely implemented the iterator using Walk from ignore crate. Let me know your thoughts @TLATER @mightyiam If everything looks good to you guys, I will add tests.

sinrohit-desco avatar Aug 19 '25 12:08 sinrohit-desco

Sorry, I think #138 is the priority so we've been working on that.

mightyiam avatar Sep 13 '25 07:09 mightyiam

Hey, #138 is complete. Sorry for the conflicts. Looking forward to reviewing this again.

mightyiam avatar Oct 14 '25 05:10 mightyiam

Hey, #138 is complete. Sorry for the conflicts. Looking forward to reviewing this again.

Hi Shahar! I have rebased the PR and resolved the conflicts.

sinrohit-desco avatar Oct 15 '25 08:10 sinrohit-desco