Add nodurations linter

[Karthik-K-N]

Fixes: #24

This PR contains most of the changes similar to nobool linter, May be I can work on code reuse based on the feedback.

[JoelSpeed]

Thanks for taking a pass at this one, I've left a few comments, pretty much all around documentation wording. If we can get those updated I think we can get this merged pretty quickly

[JoelSpeed]

We have our own internal inspector wrapper now, we should use this instead

Can you copy the pattern in

kube-api-linter/pkg/analysis/duplicatemarkers/analyzer.go

Lines 46 to 57 in 81d935c

	inspect, ok := pass.ResultOf[inspector.Analyzer].(inspector.Inspector)
	if !ok {
	return nil, kalerrors.ErrCouldNotGetInspector
	}
	inspect.InspectFields(func(field *ast.Field, _ []ast.Node, _ extractjsontags.FieldTagInfo, markersAccess markers.Markers) {
	checkField(pass, field, markersAccess)
	})
	inspect.InspectTypeSpec(func(typeSpec *ast.TypeSpec, markersAccess markers.Markers) {
	checkTypeSpec(pass, typeSpec, markersAccess)
	})

[Karthik-K-N]

Yeah, I am trying to reuse this, but finding bit confused and trying to find a way.

inspect.Preorder(nodeFilter, func(node ast.Node) {
		switch n := node.(type) {
		case *ast.StructType:
			if n.Fields == nil {
				return
			}

			for _, field := range n.Fields.List {
				checkField(pass, field)
			}
		case *ast.Field:
			checkField(pass, n)
		case *ast.TypeSpec:
			checkTypeSpec(pass, n, node, "type")
		}

1. with using inspect.InspectTypeSpec I can't get the node object to report node.Pos()
2. Trying to find way to iterate over struct fields in case of *ast.StructType

[JoelSpeed]

I can't get the node object to report node.Pos()

An *ast.TypeSpec implements the ast.Node interface, so you can substitute node with the typeSpec no?

[JoelSpeed]

Trying to find way to iterate over struct fields in case of *ast.StructType

The InspectFields handles this for you by passing in the field of any relevant struct, so you don't need to handle structs directly

[Karthik-K-N]

Got it, I just overlook few things in hurry, Just fixing final round of errors.

[Karthik-K-N]

Updated and squashed the commits.

[JoelSpeed]

Aside from https://github.com/kubernetes-sigs/kube-api-linter/pull/143/files#r2330102093, LGTM, can you pickup this comment and squash into a single commit please?

[Karthik-K-N]

Aside from https://github.com/kubernetes-sigs/kube-api-linter/pull/143/files#r2330102093, LGTM, can you pickup this comment and squash into a single commit please?

Done, Thank you for the help.

[JoelSpeed]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JoelSpeed, Karthik-K-N

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