use TypeAwareMarkerCollectionForField to collect markers

[sivchari]

close #119

/cc @sbueringer @JoelSpeed

[JoelSpeed]

Please also add a case where the field is referencing an alias type, with the marker, from another file on disk, that's important to check is working as we don't always reference types from within the same file.

[sivchari]

@JoelSpeed

Thanks, done.

[sbueringer]

@sivchari Sorry, didn't look into the details. Could it be that we can fix this issue (#113) in the optionalfields linter the same way?

[JoelSpeed]

@sivchari Sorry, didn't look into the details. Could it be that we can fix this issue (#113) in the optionalfields linter the same way?

#121 is fixing #113 in the same way, and just needs some edits to the tests to be able to merge

[JoelSpeed]

/approve
/lgtm

I think lets tackle the enum thing separately

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JoelSpeed, sivchari

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 [JoelSpeed]

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

[sbueringer]

I think lets tackle the enum thing separately

Sure absolutely. Also wasn't aware of the other PR