search

Vauxoo / pre-commit-vauxoo

pre-commit-vauxoo python library to add a command to use all the configuration files and environment variables of Vauxoo
Other
2 stars 2 forks source link

pre-commit-vauxoo: Apply it to the desired module only #4

Closed rolandojduartem closed 2 years ago

rolandojduartem commented 2 years ago

Hi, @moylop260,

I read that #2 was merged and now we can use it if we are in the repository's root or a specific folder of the project, but, the main idea is applying the pre-commit-vauxoo to a specific module, not all of them. pre-commit base use --files for this purpose. I do not know if this is part of point 2 or 3 of #1, but I think this a good feature to be considered.

Expected behaviour (pre-commit base):

Screenshot from 2022-08-19 08-50-42

Kind regards.

cc @luisg123v

moylop260 commented 2 years ago

@rolandojduartem

In your screenshot we can not see the first command than you ran

But pre-commit is getting different results using this way

I mean, running pre-commit run --all is detecting a lint check in the module branch_accounting_report

But running pre-commit run --files branch_accounting_report/ even pre-commit run --files branch_accounting_report/* is not detecting any lint:

moylop260 commented 2 years ago

I found a way to get all the files and send all them manually using --files file1, file2, file3...

This is my proposal

Could you test it, please?

Summary: