Introduce command that combines `check` and `detect-stale-violations` about packwerk

[rafaelfranca]

Do we need to change or add a new command command? Can't you also run packwerk detect-stale-violations on CI?

[mclark]

Yes, we can definitely run the other check as a separate CI but it just seems wasteful to me. At scale, that's a lot of processing that doesn't really need to happen, we can probably make a worthwhile positive environmental impact by combining these two.

[rafaelfranca]

I agree, but at the same time if we combine both commands in our CI it would take more time than running all tests of the app, I'm surprised this is not the case in your app as well.

I'm ok with the inclusion of a new command though, but I'd prefer to change check to do both. Can you see how much the total time would increase if we change check to do both? If it is a lot we will need to create another command.

[mclark]

but at the same time if we combine both commands in our CI it would take more time than running all tests of the app, I'm surprised this is not the case in your app as well.

My thinking is that the parse will only have to happen once, meaning the call to stale_violations? will be the only addition to check and is relatively quick. That being said, I actually haven't verified that yet and may be overlooking something. I'll set up a branch and let you know.