feat(conformance): validate implementation flags
What type of PR is this?
/area conformance-machinery
What this PR does / why we need it:
The conformance suite implementation-related flags are properly validated.
Which issue(s) this PR fixes:
Fixes #2178
Does this PR introduce a user-facing change?:
The conformance suite implementation-related flags are properly validated.
/retest
/lgtm
/retest
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mlavacca
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [mlavacca]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
@youngnick I lost your lgtm as I needed to lint the code. Do you mind taking another look?
/lgtm
Bit odd to have a function called DefaultOptions that has required flags to be set in it.