Update exhaustivestruct

[mbilski]

fixes:

• ignore private fields
• report multiple missing fields at once

[bombsimon]

Generally looks good but how do you feel about tests? Of course the heavy lifting and majority of testing should be a part of the linter but I think it's good to cover at least one of each pass in this repo too. The reason being that a too big change to the linter hidden behind a change to only go.mod could have unexpected affect to the end user. By always ensuring the testdata adds or removes passes it's easier to see how big the change is.

Wdyt? Should we add tests for the changes you did such as ignoring private fields or do you think the single pass in the current testdata is enough?

[mbilski]

Generally looks good but how do you feel about tests? Of course the heavy lifting and majority of testing should be a part of the linter but I think it's good to cover at least one of each pass in this repo too. The reason being that a too big change to the linter hidden behind a change to only go.mod could have unexpected affect to the end user. By always ensuring the testdata adds or removes passes it's easier to see how big the change is.

Wdyt? Should we add tests for the changes you did such as ignoring private fields or do you think the single pass in the current testdata is enough?

makes sense, I'll add some tests

[bombsimon]

Thanks, LGTM!