fix: local overcommit file merges with existing .overcommit.yml file#815
fix: local overcommit file merges with existing .overcommit.yml file#815sds merged 1 commit intosds:masterfrom
Conversation
c97ec56 to
be113dd
Compare
|
@sds Greetings! I was using the feature I added earlier this year and realized it had a bug. Providing a fix here for your consideration. |
|
I just hit the same bug and found it rather confusing. Anything I can do to help get this merged? |
👋🏻 @simoleone , I need to take a look at where I left this. If it works as I had intended we can tag the maintainers for review I think. I'll try to get to it this weekend! If you'd like to test this branch locally as well and make sure it works as expected for the issue you found as well, that would probably be great. |
|
hey! Yup this works the way I think it should 😄 |
|
Hey! Any way I can help out with this? 😄 . I know how free time goes with my own projects (or doesn't 🙃 ). |
Hi! Ah so sorry, yes, time is hard to find 🙃 |
The .local-overcommit.yml did not work as intended. When a repo level .overcommit.yml and .local-overcommit.yml both existed, only the latter applied. Further, the tests only checked that each file was loaded, not that the configs included the expected overrides.
This PR refactors: