courtney icon indicating copy to clipboard operation
courtney copied to clipboard

Published 20 hours ago • verified publisher icon dave

Exclude generated code from coverage

Open breml opened this issue 2 years ago • 4 comments

Fixes #15

breml avatar Mar 28 '22 13:03 breml

Should there be a flag/config option for this? There could be projects interested in coverage of generated code.

imsodin avatar Mar 29 '22 13:03 imsodin

Yeah I think running tests on generated code is common... I do it in some of my projects.

dave avatar Mar 29 '22 13:03 dave

@dave No problem, I can add a flag for this. Do you have any preference on the name of the flag? I assume, the default would be to keep the existing behavior for backwards compatibility reasons.

breml avatar Mar 29 '22 14:03 breml

@dave Another question, there are several places, where gofmt (respectively goimports and gofumpt) would change the formatting of the code. I did not commit these changes so far. How do you feel about this?

breml avatar Mar 29 '22 14:03 breml