disable unions, since they're still experimental

[lavalamp]

benchmark                       old ns/op     new ns/op     delta
BenchmarkApplyNewObject-12      1355981       1363050       +0.52%
BenchmarkUpdateNewObject-12     1565341       1228947       -21.49%
BenchmarkRepeatedUpdate-12      443528        396604        -10.58%
BenchmarkSetToFields-12         5154          5107          -0.91%
BenchmarkFieldsToSet-12         10050         10170         +1.19%
BenchmarkYAMLToTyped-12         56860         54439         -4.26%

benchmark                       old allocs     new allocs     delta
BenchmarkApplyNewObject-12      4264           4214           -1.17%
BenchmarkUpdateNewObject-12     4850           3661           -24.52%
BenchmarkRepeatedUpdate-12      1255           1179           -6.06%
BenchmarkSetToFields-12         14             14             +0.00%
BenchmarkFieldsToSet-12         82             82             +0.00%
BenchmarkYAMLToTyped-12         90             90             +0.00%

[apelisse]

Hmm, not sure why you did it this way, I don't really want to have this for a long time, wouldn't it have been easier to jusk skip the tests?

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: apelisse, lavalamp

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [apelisse,lavalamp]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[apelisse]

Actually, you forgot one just below (yes, we call it before AND after merging)

[lavalamp]

#98

[lavalamp]

Didn't just skip the tests because if the code is checked in, it needs to not bitrot. Otherwise I need to delete it.

[apelisse]

That's fair